Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

Better handling of buttons position in mobile view #349

Closed

Conversation

GuillaumeGomez
Copy link
Contributor

No description provided.

Copy link
Contributor

@amatissart amatissart left a comment

Choose a reason for hiding this comment

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

This solution looks a bit hackish to me.
I would prefer to find a way to remove dynamic css completely.

Or at least, we should try to refactor how these buttons are positioned all over the application.
Some css classes are already updated dynamically on buttons in road_map_panel.js and direction_panel.js.
This change should be a opportunity to handle that in a single place.

Anyway, I would wait for @bbecquet's opinion.

src/main.js Outdated
el.styleSheet.cssText = css;
} else {
el.appendChild(document.createTextNode(css));
}
Copy link
Contributor

Choose a reason for hiding this comment

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

this looks very complicated.
Why not el.innerHTML = css ?

@amatissart
Copy link
Contributor

Implemented by #365

@amatissart amatissart closed this Sep 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants