Skip to content

dbeaver/pro#9877 UI fixes#4473

Open
SychevAndrey wants to merge 11 commits into
develfrom
9877-cb-fix-ui-styles-2
Open

dbeaver/pro#9877 UI fixes#4473
SychevAndrey wants to merge 11 commits into
develfrom
9877-cb-fix-ui-styles-2

Conversation

@SychevAndrey

Copy link
Copy Markdown
Contributor

No description provided.

@SychevAndrey SychevAndrey self-assigned this Jul 20, 2026
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity

Metric Results
Complexity 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

readOnly={readonly}
description={property.hint}
className={className}
portal={true}

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.

Could break the layout when the selector is shown inside a form and the page is scrolled. The selector may overlap surrounding elements. Also please eslint rule, just porta={true} -> portal

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

But it is also breaking without it. So, what are we going to do?

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.

@sergeyteleshev could you please check the case with broken layout that you ve fixed several month ago?

@SychevAndrey
SychevAndrey requested a review from devnaumov July 21, 2026 09:35
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