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

SelectComponent does not render droplist correctly in dialog (#12895) #13261

Merged
merged 3 commits into from
Jan 11, 2024

Conversation

dhuebner
Copy link
Member

@dhuebner dhuebner commented Jan 11, 2024

What it does

Fixes #12895

How to test

I've added a new "Sample React Dialog" as there were any samples that are using react. Open it using the "Sample React Dialog" command.

Before:
Bildschirmfoto 2024-01-11 um 10 57 45

After:

Bildschirmfoto 2024-01-11 um 10 56 27

Follow-ups

I can remove the sample dialog again if you don't want to have it.
Also, I don't know why, but the *.css file was auto formatted with a probably different rules as it was initially created.

@msujew msujew added the ui/ux issues related to user interface / user experience label Jan 11, 2024
Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

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

Thanks, the change looks good to me 👍

One minor issue, see below. Afterwards, this is good to go.

@dhuebner
Copy link
Member Author

@msujew
Thank you Mark. Do you know why the git tests are failing?

@msujew
Copy link
Member

msujew commented Jan 11, 2024

@dhuebner You need to adjust the references to the old SampleReactDialog, which was renamed.

@msujew msujew merged commit 160dce9 into eclipse-theia:master Jan 11, 2024
14 checks passed
@dhuebner dhuebner deleted the huebner/selectComponent-12895 branch January 11, 2024 13:24
@jfaltermeier jfaltermeier added this to the 1.46.0 milestone Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui/ux issues related to user interface / user experience
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

SelectComponent does not render droplist correctly when embedded in a dialog
3 participants