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

tablereport display details #1067

Merged
merged 17 commits into from
Sep 23, 2024
Merged

Conversation

jeromedockes
Copy link
Member

a small detail so it is aligned when wrapping

@Vincent-Maladiere
Copy link
Member

It's hard to see what changes, can you provide screenshots?

@jeromedockes
Copy link
Member Author

sure, it's really a detail. it is just about alignment of the dropdown when there is a very narrow display

before:

screenshot_2024-09-12T13:52:34+02:00

after:

screenshot_2024-09-12T13:54:53+02:00

@jeromedockes
Copy link
Member Author

the change on the tab margin is just that on a high-resolution large screen at some zoom levels in jupyter notebooks only I could see a very thin gray line between the selected tab and its panel, but I can't seem to reproduce it on my laptop to provide a screenshot

@jeromedockes jeromedockes changed the title remove margin of column filter select tablereport display details Sep 16, 2024
@jeromedockes jeromedockes marked this pull request as draft September 16, 2024 12:28
@jeromedockes jeromedockes added this to the 0.3.1 milestone Sep 23, 2024
@jeromedockes jeromedockes marked this pull request as ready for review September 23, 2024 16:24
@Vincent-Maladiere
Copy link
Member

Hey @jeromedockes, is this PR ready for review?

@jeromedockes
Copy link
Member Author

jeromedockes commented Sep 23, 2024 via email

Copy link
Contributor

@TheooJ TheooJ left a comment

Choose a reason for hiding this comment

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

Thanks !

@TheooJ TheooJ merged commit 6bacc73 into skrub-data:main Sep 23, 2024
22 checks passed
@jeromedockes jeromedockes deleted the report-minor-detail branch September 24, 2024 07:30
@Vincent-Maladiere
Copy link
Member

Well, I would have appreciated reviewing it before merging.

jeromedockes added a commit to jeromedockes/skrub that referenced this pull request Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants