Skip to content

feat(Switcher): Improve mobile layout#310

Merged
sjockers merged 5 commits into
mainfrom
feat/switcher-layout-mobile
Feb 6, 2026
Merged

feat(Switcher): Improve mobile layout#310
sjockers merged 5 commits into
mainfrom
feat/switcher-layout-mobile

Conversation

@sjockers
Copy link
Copy Markdown
Contributor

@sjockers sjockers commented Feb 4, 2026

Two tiny improvements, requested while testing new Datawrapper switcher:

image

@sjockers sjockers requested a review from awesomephant February 4, 2026 13:23
@sjockers sjockers changed the title feat(switcher): Improve mobile layout feat(Switcher): Improve mobile layout Feb 4, 2026
Copy link
Copy Markdown
Collaborator

@awesomephant awesomephant left a comment

Choose a reason for hiding this comment

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

layout: "row" | "column" | "auto" statt forceRow?

Copy link
Copy Markdown
Collaborator

@awesomephant awesomephant left a comment

Choose a reason for hiding this comment

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

Kann man das ohne @mixin implementieren? Dupliziert code + stärkt sass dependency

@sjockers
Copy link
Copy Markdown
Contributor Author

sjockers commented Feb 4, 2026

layout: "auto" hat jetzt technisch gesehen keine Funktion (default styling), ich habe es trotzdem drin gelassen um die möglichen Settings expliziter zu machen.

@sjockers sjockers requested a review from awesomephant February 4, 2026 15:23
@sjockers sjockers force-pushed the feat/switcher-layout-mobile branch from 782aae5 to 005762f Compare February 4, 2026 15:33
@sjockers sjockers merged commit 7b088b1 into main Feb 6, 2026
1 check passed
@sjockers sjockers deleted the feat/switcher-layout-mobile branch February 6, 2026 11:34
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