-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
One Omnibar Widget #13
Comments
I put in an enhancement request for this consolidation. |
…ase/1.3.1 * commit '267e14b4005770099467d94e424fd83fbc08a2fa': Fix bug where the omnibar does not work properly on strict servers.
This could be an addition to the exisiting omni bar widget, but getting rid of the omni bar would be a breaking change for users currently using this widget. Backwards compatibility should be considered here. |
I agree backwards compatibility is an issue. That is why I suggest it is done in stages. Stage 1: Stage 2: Stage 3: |
@bhubbard I added the omnibar label. |
I think its more confusing to have two omnibar widgets. We should just have just one omnibar widget, with a checkbox option in the widget to display the extra fields.
Even better if we make it a checkbox option for each field we might want to display. Example:
The text was updated successfully, but these errors were encountered: