Opened 10 years ago

Closed 10 years ago

#8707 closed bug (wontfix)

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

Change History (5)

comment:1 Changed 10 years ago by Scott González

Component: ui.coreui.menu
Owner: set to crowcoder
Status: newpending

Why are you putting buttons inside menus?

comment:2 in reply to:  1 Changed 10 years ago by crowcoder

Status: pendingnew

Replying to 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.

comment:3 Changed 10 years ago by Scott González

Status: newpending

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

comment:4 in reply to:  3 Changed 10 years ago by crowcoder

Status: pendingnew

Replying to 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.

comment:5 Changed 10 years ago by Scott González

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.

Note: See TracTickets for help on using tickets.