Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Better handling of iconpos on navbars #5374

Closed
ucavus opened this issue Dec 14, 2012 · 5 comments
Closed

Better handling of iconpos on navbars #5374

ucavus opened this issue Dec 14, 2012 · 5 comments

Comments

@ucavus
Copy link

ucavus commented Dec 14, 2012

Currently, the icon position for every button in a navbar must be in the same position. This seems clunky and it is unnecessary to have this restriction. The current CSS can render a mixture of left, right and not set, or top and bottom, perfectly well without any changes. The attached images show the result of manually switching and removing ui-btn-icon-* classes in the DOM.

The enhancement functions could be changed to allow such overrides per navbar item. A little extra CSS you could even have a mix of horizontal and vertical positions, though I can't imagine that looking very good.

Even if this improvement is not implemented, it would be good if the classes ui-btn-icon-right or ui-btn-icon-left were not added to items that have no icon. The classes make the text-only buttons look misaligned because of the extra padding.

navbar-multiple-iconpos-1
navbar-multiple-iconpos-2

@jaspermdegroot
Copy link
Contributor

@ucavus

Thanks for your post. I agree that we can improve the way icon position is handled in the navbar widget.
For the 1.4 release we are planning on re-factoring the code and will look into this issue as well.

@jaspermdegroot
Copy link
Contributor

We should also make it possible to have navbar buttons with icons only. http://jsbin.com/exigay/5/edit

@Whip74
Copy link

Whip74 commented Jul 13, 2013

@uGoMobi

Are there any temporary solutions to icon positioning within the navbar widget until jquery version 1.4 is released? I have a simple two button navbar and need the icons to fall to the left on both buttons.

@ucavus
Copy link
Author

ucavus commented Jul 15, 2013

Just attach a function to pageinit and change the css classes manually.

@jaspermdegroot
Copy link
Contributor

Closing as duplicate of #6337.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants