Since Image Assist (currently?) heavily depends on Image module, I see no reason why it shouldn't place its settings page on admin/settings/image/img_assist

Would look nicer in admin menu?

CommentFileSizeAuthor
#6 img_assist_move_menu_item.patch2.3 KBCloudCuckoo

Comments

CloudCuckoo’s picture

Yes please! I always expect the image assist settings to be there anyway.

Trivial patch attached as my +1.

CloudCuckoo’s picture

Well it would've been if it was letting me upload.

sun’s picture

Version: 6.x-2.x-dev » 6.x-3.x-dev

Please try again now. Also, patch should be against 3.x.

twod’s picture

Status: Active » Needs review

I like this idea and made a patch on my own.
It makes the menu item an "Image Attach"-tab on the Images page instead. (Same location as suggested above, just no visible meny link.)

It also takes care of a drupal_goto() call, a switch case in hook_help() and updates the readme.

sun’s picture

Status: Needs review » Active

Unfortunately, your patch did not make it through

CloudCuckoo’s picture

Status: Active » Needs review
StatusFileSize
new2.3 KB

Here's another go.

Patch does the same as #4 except the tab title is Image assist, to avoid any confusion with the Image attach module.

twod’s picture

Cloudy's patch is identical to mine so I won't bother re-attaching it.

sun’s picture

On a second thought, and taking #175812: Imagefield support into account, I am no longer sure whether this is a good idea. If Image assist is able to work without Image module (ImageField instead) in the near future, why should its settings be located under Image module's settings?

sun’s picture

Status: Needs review » Closed (won't fix)

Given D7 features, this is definitely not a good idea anymore.