Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: More tweaks needed after adding Doctype tag #10504

Merged
merged 8 commits into from
Aug 13, 2020

Conversation

rusackas
Copy link
Member

@rusackas rusackas commented Aug 3, 2020

SUMMARY

Mainly, fixes a bug with the FilterBox in the Explore view. Also consolidates a few bits of styling, and adds 'px' to some font size styles that should have had them in the first place.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

After screenshots, now working:
image

image
image

TEST PLAN

ADDITIONAL INFORMATION

@rusackas rusackas requested a review from ktmud August 3, 2020 05:04
@rusackas rusackas changed the title More tweaks needed after adding Doctype tag fix: More tweaks needed after adding Doctype tag Aug 3, 2020
@rusackas
Copy link
Member Author

Fixes #10589

@rusackas rusackas merged commit a3fd2b4 into apache:master Aug 13, 2020
@rusackas rusackas deleted the doctype-round2 branch August 13, 2020 18:50
@mistercrunch mistercrunch added the change:frontend Requires changing the frontend label Aug 13, 2020
Ofeknielsen pushed a commit to ofekisr/incubator-superset that referenced this pull request Oct 5, 2020
* fixing last-of-type specificity

* Simplifying/consolidating styles

* fixing filterbox in explore

* linting

* adding some px to font sizes

* fixing in-dash overflow

* removing px for one special fontsize case

* simplifying, addressing comments
auxten pushed a commit to auxten/incubator-superset that referenced this pull request Nov 20, 2020
* fixing last-of-type specificity

* Simplifying/consolidating styles

* fixing filterbox in explore

* linting

* adding some px to font sizes

* fixing in-dash overflow

* removing px for one special fontsize case

* simplifying, addressing comments
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.38.0 labels Mar 12, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels change:frontend Requires changing the frontend size/M 🚢 0.38.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Filter box chart does not show any rows
4 participants