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

"Merkliste teilen" – text and layout change #226

Open
owlRike opened this issue Oct 24, 2020 · 1 comment · May be fixed by #186
Open

"Merkliste teilen" – text and layout change #226

owlRike opened this issue Oct 24, 2020 · 1 comment · May be fixed by #186
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@owlRike
Copy link

owlRike commented Oct 24, 2020

Text change to:
Du kannst Deine Merkliste mit Deinen Freunden, Bekannten und
der ganzen Welt über Social Media teilen.

Aktuell ist Deine Liste nicht geteilt.

Make the space between the text and the buttons bigger

Buttons:
_ make both buttons the same size
_ make both buttons the same colour like the "schließen" button
Bildschirmfoto 2020-10-24 um 15 27 56

@owlRike owlRike added the enhancement New feature or request label Oct 24, 2020
@christianaschoff
Copy link
Contributor

making the buttons the same color conflicts with the style of the whole application. Same as it was on the trader overview page.
The darker green represents the currently focused element, so if you would hit enter on a desktop, this is the action that is being executed.
Its Material UIs default behaviour.

@christianaschoff christianaschoff self-assigned this Oct 25, 2020
@christianaschoff christianaschoff added this to the Release-7 milestone Oct 25, 2020
@christianaschoff christianaschoff linked a pull request Oct 25, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants