Skip to content

feat(design-system): change DsDrawer responsive strategy [AR-53856]#358

Closed
vpolessky-dn wants to merge 1 commit intodrivenets:nextfrom
vpolessky-dn:feat/AR-53856-change-drawer-responsive
Closed

feat(design-system): change DsDrawer responsive strategy [AR-53856]#358
vpolessky-dn wants to merge 1 commit intodrivenets:nextfrom
vpolessky-dn:feat/AR-53856-change-drawer-responsive

Conversation

@vpolessky-dn
Copy link
Copy Markdown
Collaborator

@vpolessky-dn vpolessky-dn commented Apr 10, 2026

No description provided.

…nsive styling [AR-53842] (drivenets#338)

feat(design-system): change DsDrawer responsive strategy [AR-53856]

Made-with: Cursor
@vpolessky-dn vpolessky-dn changed the title feat(design-system): create reusable helper function to support respo… feat(design-system): create reusable helper function to support responsive styling, change DsDrawer responsive strategy [AR-53856] Apr 10, 2026
@vpolessky-dn vpolessky-dn changed the title feat(design-system): create reusable helper function to support responsive styling, change DsDrawer responsive strategy [AR-53856] feat(design-system): change DsDrawer responsive strategy [AR-53856] Apr 10, 2026
width: var(--col-span-#{$i});
}
}
}
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.

Why are you removing this? It should be the same as per what I'm seeing in Figma

Am I missing something?

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.

  1. This PR is built on top of something else? Why are all the changes in non-drawer code?
  2. This button should be removed in next. I don't think we should bother with it 🤷

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.

what are these changesets names? it should be auto-generated by pnpm run changelog, not written manually

@vpolessky-dn
Copy link
Copy Markdown
Collaborator Author

@StyleShit moving changes to another PR

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.

2 participants