Skip to content

Conversation

@GretaD
Copy link
Contributor

@GretaD GretaD commented Nov 5, 2019

ref #1740

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Run npm run lint:autofix :)

@GretaD GretaD force-pushed the bugfix/1740/fix_visibility_mobile branch 3 times, most recently from 02e44f1 to e07188f Compare November 6, 2019 11:26
@ChristophWurst
Copy link
Member

Run npm run lint:autofix :)

Try again 🙊

Signed-off-by: Greta Doci <gretadoci@gmail.com>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@jancborchardt
Copy link
Member

@GretaD and @violoncelloCH did you coordinate on this then? Is this PR or #2222 going to be used going forward? :)

@violoncelloCH
Copy link
Member

as also discussed with @juliushaertl and @skjnldsv the pure css approach in #2222 seems to be the better way to do this (less performance heavy as it just uses css flex)

@skjnldsv skjnldsv closed this Nov 10, 2019
@ChristophWurst ChristophWurst deleted the bugfix/1740/fix_visibility_mobile branch January 17, 2023 16:50
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.

6 participants