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

Improve usage readability #357

Closed

Conversation

pixelbrackets
Copy link
Contributor

@pixelbrackets pixelbrackets commented Sep 9, 2024

Make usage messages more readable (like “48.476 of 5.000.000” instead of “48476 / 5000000”).

Also adapt the action icon SVG similar to existing action icons of the TYPO3 core (these dont set fixed colors in order to let them change as needed in the context). Right now the DeepL icon is the only one in the system information bar which is black, this PR will fix this.

Before
Bildschirmfoto vom 2024-09-06 16-58-53

After
Bildschirmfoto vom 2024-09-06 21-28-28

@pixelbrackets pixelbrackets changed the title Usage readability Improve usage readability Sep 9, 2024
@calien666
Copy link
Member

@pixelbrackets Please rebase your branch to the latest main, as you forgot allow editing by us and I can't do. If you have done, the pipelines will succeed.

Action icons dont have a fixed color set, in order to let TYPO3 set
the desired color - black in list actions, white in toolbar infos.

Change the DeepL SVG to not use black in all places.
@pixelbrackets
Copy link
Contributor Author

pixelbrackets commented Oct 21, 2024

@calien666 »Allow edit by maintainers« was/is active already.

Branch is rebased and green again :)

@pixelbrackets
Copy link
Contributor Author

pixelbrackets commented Nov 5, 2024

@calien666 The PR is missing one approving review yet - are you able to do this?

@calien666 calien666 requested a review from NarkNiro November 6, 2024 14:30
@calien666
Copy link
Member

Done by #358

Thanks for your work

@calien666 calien666 closed this Nov 20, 2024
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