Skip to content

Fix license table CSS selector to apply the selected row styles#16547

Merged
jtpio merged 1 commit into
jupyterlab:mainfrom
joaopalmeiro:fix-color-contrast-license-table
Jul 3, 2024
Merged

Fix license table CSS selector to apply the selected row styles#16547
jtpio merged 1 commit into
jupyterlab:mainfrom
joaopalmeiro:fix-color-contrast-license-table

Conversation

@joaopalmeiro

Copy link
Copy Markdown
Contributor

Hi! 👋

References

Closes #16542

Code changes

Standardize the license table CSS selector to .jp-Licenses-Grid.jp-RenderedHTMLCommon instead of just .jp-Licenses-Grid so that selected row styles are applied as expected.

Note

I had to manually copy the third-party-licenses.json files to the ~/jupyterlab/dev_mode/static/ folder in development mode in order to test the changes. This file is not generated in development mode as far as I noticed.

User-facing changes

The text of the selected row is visible, similar to what happens in the file browser:

Screenshot 2024-07-02 at 19 23 32

Backwards-incompatible changes

None, as far as I know.

@jupyterlab-probot

Copy link
Copy Markdown

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@github-actions github-actions Bot added pkg:help tag:CSS For general CSS related issues and pecadilloes Design System CSS labels Jul 2, 2024
@jtpio jtpio added the bug label Jul 3, 2024

@jtpio jtpio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@jtpio jtpio added this to the 4.2.x milestone Jul 3, 2024
@jtpio jtpio merged commit 654fa12 into jupyterlab:main Jul 3, 2024
@jtpio

jtpio commented Jul 3, 2024

Copy link
Copy Markdown
Member

@meeseeksdev please backport to 4.2.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Jul 3, 2024
@joaopalmeiro joaopalmeiro deleted the fix-color-contrast-license-table branch July 3, 2024 09:53
krassowski pushed a commit that referenced this pull request Jul 3, 2024
…ted row styles (#16550)

Co-authored-by: João Palmeiro <joaommpalmeiro@gmail.com>
ImpSy pushed a commit to spotinst/jupyterlab that referenced this pull request Jan 7, 2025
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Design System CSS pkg:help tag:CSS For general CSS related issues and pecadilloes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lack of color contrast in the license selected in Help > Licenses

2 participants