Hi, I'd like to ask if a merge (I mean, grouping under the same sub-menu) of ImageAPI settings and (Drupal) ImageToolkit could be a doable task.

First-time users should have one place to set image settings, and with ImageAPI few more tabs in the same menu.

What do you think?

CommentFileSizeAuthor
#5 Imagetoolkit_1239973931368.png12.02 KBthepanz
#5 Imagetoolkit_1239973972524.png17.03 KBthepanz
#5 Imagetoolkit_1239974059040.png12.79 KBthepanz
#5 Imagetoolkit_1239974072736.png9.62 KBthepanz
#5 Imagetoolkit_1239974093882.png15.69 KBthepanz

Comments

drewish’s picture

drewish commented

I agree that it's confusing but I don't think we really can. Core's image settings require .inc files and ImageAPI's require modules. So you could have the ImageMagick module installed but not a .inc toolkit. If you selected ImageMagick then ImageAPI would work correctly but core's image operations would not.

thepanz’s picture

thepanz commented

@drewish: I was talking about Menu entry/position, not system functionality! :)

What I mean is to place:

Image Toolkit -> admin/settings/image-toolkit (as 1st level Default Local Task)
ImageApi LIST -> admin/settings/image-toolkit/imageapi (2nd level Default Local Task)
ImageApi Settings -> admin/settings/image-toolkit/imageapi/settings (2nd level Local Task)

What do you think?

drewish’s picture

drewish commented

ah, that makes more sense. i'm not sure how it would work in practice but if you would mind whipping up a quick patch i'd love to see what you're describing.

thepanz’s picture

thepanz commented

@drewish: ok :) I'll let you know! give me 24/48 hrs! :)

thepanz’s picture

thepanz commented
StatusFileSize
new Imagetoolkit_1239974093882.png15.69 KB
new Imagetoolkit_1239974072736.png9.62 KB
new Imagetoolkit_1239974059040.png12.79 KB
new Imagetoolkit_1239973972524.png17.03 KB
new Imagetoolkit_1239973931368.png12.02 KB

Hi, I can't provide a patch right now (CVS external access is disabled at my University).. I've made some screenshost with ImageAPI under ImageToolkit admin menu (with one and two toolkits installed)..

What do you think?

Note: if only one toolkit is installed, Configure will appear as title for configuring it;
else if two or more toolkits are present, a new tab for each config will appear..

drewish’s picture

drewish commented

humm... i see what you're saying but i'm not sure that makes it much easier. people often don't see the tabs much less the secondary tabs.

thepanz’s picture

thepanz commented

I think that Drupal admins (the real users of Image-API settings IMHO) are quite comfortable with primary/secondary tabs :)
In addition, you can add, after module install, a message like "ImageAPI successfully installed, visit _LINK_ for configuration"..

I think that moving ImageAPI into ImageToolkit menu can group the "imaging" functionality in one single place, so admins will not have another entry in /admin/settings and can easily configure ImageToolkit and ImageAPI with only one click in the tabs instead of go back to admin/settings and look for ImageAPI..

What do you think? Maybe other admins/users can give their opinion... :)

Cheers

thepanz’s picture

thepanz commented
Status: Active » Needs review
drewish’s picture

drewish commented
Status: Needs review » Active

There's no patch.