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.

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

[Dataset page] Fix displaying filter button label #64

Merged
merged 2 commits into from
May 7, 2024
Merged

Conversation

tkohr
Copy link
Member

@tkohr tkohr commented May 2, 2024

PR fixes displaying the button label using a mel tailwind component instead of gn-ui button (whose css somehow gets overriden by tailwind base button css). It also aligns the layout for this button with the mel badge layout (as mel buttons are quite big here).

Also moves the google fonts to resources/assets that already provides shared assets for the apps.

using a mel tailwind component instead of gn-ui button (whose css gets overriden by tailwind base button css)
it also aligns the layout fot his button with the mel badge layout (as mel buttons are quite big here)
@tkohr tkohr force-pushed the fix-download-filter branch from cbbf9fa to 8a55488 Compare May 2, 2024 13:03
Copy link
Contributor

@Angi-Kinas Angi-Kinas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @tkohr LGTM 👍
I checked it locally and it looks fine :-)

@tkohr tkohr merged commit 9329a0f into main May 7, 2024
7 checks passed
@tkohr tkohr deleted the fix-download-filter branch May 7, 2024 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants