Skip to content
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.

Sign up for GitHub

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

Jump to bottom

WARNING_MSG conflicts with SQL Lab #1308

Closed
0x0ece opened this issue Oct 10, 2016 · 1 comment
Closed

WARNING_MSG conflicts with SQL Lab #1308

0x0ece opened this issue Oct 10, 2016 · 1 comment

Comments

@0x0ece
Copy link
Contributor

0x0ece commented Oct 10, 2016

If WARNING_MSG is set in the config, the view in SQL Lab breaks, meaning that the header overlaps with the warning message.

Maybe WARNING_MSG can be moved into the very top header, between the menu and the right icons?
@ascott
Copy link
Contributor

ascott commented Oct 11, 2016

i've noticed that too, fix here: #1312

@0x0ece 0x0ece closed this as completed Jan 20, 2017
zhaoyongjie added a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 17, 2021
@zhaoyongjie
fix: add advanced analytics to all of timeseries viz (apache#1308)
56ff5f6 
* feat: add advanced analytics to all of timeseries viz

* fix namespace
zhaoyongjie added a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 24, 2021
@zhaoyongjie
fix: add advanced analytics to all of timeseries viz (apache#1308)
c090699 
* feat: add advanced analytics to all of timeseries viz

* fix namespace
zhaoyongjie added a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 25, 2021
@zhaoyongjie
fix: add advanced analytics to all of timeseries viz (apache#1308)
d8ab6da 
* feat: add advanced analytics to all of timeseries viz

* fix namespace
zhaoyongjie added a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 26, 2021
@zhaoyongjie
fix: add advanced analytics to all of timeseries viz (apache#1308)
4de4fc3 
* feat: add advanced analytics to all of timeseries viz

* fix namespace
@gitworkflows gitworkflows mentioned this issue Jan 18, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ascott @0x0ece