Skip to main content

Search and Top Navigation

#8707 closed bug (wontfix)

Opened October 22, 2012 12:44PM UTC

Closed October 22, 2012 02:39PM UTC

jQuery icons do not center on buttons inside jQuery menu

Reported by: crowcoder Owned by: crowcoder
Priority: minor Milestone: 1.10.0
Component: ui.menu Version: 1.9.0
Keywords: Cc:
Blocked by: Blocking:
Description

Inside a jQueryUI menu, icons (ui-icon-trash, ui-icon-disk, etc.) do not center on buttons. They are stuck in the top left corner. The same code, outside of the menu, behaves appropriately.

I can edit jquery-ui.css to fix it by removing "top" and "left" from ui-menu ui-icon.

my jsbin:

http://jsbin.com/welcome/37845/edit

Attachments (0)
Change History (5)

Changed October 22, 2012 01:49PM UTC by scottgonzalez comment:1

component: ui.coreui.menu
owner: → crowcoder
status: newpending

Why are you putting buttons inside menus?

Changed October 22, 2012 02:06PM UTC by crowcoder comment:2

status: pendingnew

Replying to [comment:1 scott.gonzalez]:

Why are you putting buttons inside menus?

My intent was to have a little form (not literally <form>, just inputs and buttons) inside each menu item. Basically I need to list a bunch of lookup tables and allow CRUD on them. I thought the Menu with top alignment would be easier than showing/hiding divs, and better UI than modal dialogs or separate pages. I realize it is not the anticipated use of Menu.

Changed October 22, 2012 02:22PM UTC by scottgonzalez comment:3

status: newpending

This doesn't seem like a good use for menu. Do you have examples of other existing menus that function this way?

Changed October 22, 2012 02:36PM UTC by crowcoder comment:4

status: pendingnew

Replying to [comment:3 scott.gonzalez]:

This doesn't seem like a good use for menu. Do you have examples of other existing menus that function this way?

I do not. This is my own brilliant idea. It all went well except for the icon alignment. But it sounds like I should just do it a different way.

Changed October 22, 2012 02:39PM UTC by scottgonzalez comment:5

resolution: → wontfix
status: newclosed

I would assume you can fix this with a bit of CSS, but it doesn't sound like a good idea.