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

fix(Dialog): correct width's vars order #1510

Merged
merged 2 commits into from
Apr 12, 2024
Merged

fix(Dialog): correct width's vars order #1510

merged 2 commits into from
Apr 12, 2024

Conversation

saxumcordis
Copy link
Contributor

it is necessary to reverse the order of variables to give opportunity to change dialog width

it is necessary to reverse the order of variables to give opportunity to change dialog width
@saxumcordis saxumcordis linked an issue Apr 12, 2024 that may be closed by this pull request
@gravity-ui-bot
Copy link
Contributor

Preview is ready.

@gravity-ui-bot
Copy link
Contributor

Playwright Test Component is ready.

@amje amje changed the title fix: change order of vars to opportunity of changing dialog width fix(Dialog): correct width's vars order Apr 12, 2024
@@ -9,7 +9,7 @@ $block: '.#{variables.$ns}dialog';
position: relative;
display: flex;
flex-direction: column;
width: var(--_--width, var(--g-dialog-width));
Copy link
Contributor Author

@saxumcordis saxumcordis Apr 12, 2024

Choose a reason for hiding this comment

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

why here is double var()

its ok to use var(--g-dialog-width, --_--width), isn't it?

@amje

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

@saxumcordis No, @ValeraS is right, syntax is incorrect

src/components/Dialog/Dialog.scss Outdated Show resolved Hide resolved
@saxumcordis
Copy link
Contributor Author

not actual

@saxumcordis saxumcordis deleted the saxumcordis-patch-1 branch April 12, 2024 15:19
@ValeraS ValeraS restored the saxumcordis-patch-1 branch April 12, 2024 15:28
@ValeraS ValeraS reopened this Apr 12, 2024
@ValeraS ValeraS merged commit f54da2d into main Apr 12, 2024
4 checks passed
@ValeraS ValeraS deleted the saxumcordis-patch-1 branch April 12, 2024 15:37
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.

Can't change dialog width
4 participants