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

Bootstrap 5 #1379

Merged
merged 65 commits into from
Feb 14, 2024
Merged

Bootstrap 5 #1379

merged 65 commits into from
Feb 14, 2024

Conversation

jakobvogel
Copy link
Member

Description

Ports Tycho to Bootstrap 5.

Additional Notes

  • This PR fixes or works on following ticket(s): SIRI-924

Checklist

  • Code change has been tested and works locally
  • Code was formatted via IntelliJ and follows SonarLint & best practices

The library is no longer maintained and can allegedly be replaced by a bootstrap 5 method.
This makes dropdowns work again.

SIRI-924
Bootstrap 5 generally moves towards better support for Right-to-Left languages. Consequently, `left` and `right` are now replaced by `start` and `end` in the usual Left-to-Right setting. In order not to have too much confusion between frontend and backend, this commit introduces additional, similar class names in the design system.

SIRI-924
In general, Bootstrap 5 replaced `left` with `start` and `right` with `end`, in order to support Right-to-Left languages. Also includes further adaptions. See https://getbootstrap.com/docs/5.0/migration/.

SIRI-924
The original strings were of considerably different length, potentially causing the tooltip to jump.

SIRI-924
This has been forgotten to commit in 7e4f135.

SIRI-924
Previously, only #rrggbb strings have been previewed in the color picker. With this PR, shorthand, color names or other valid CSS color statements are also supported. Note, however, that the original text is sent to the server, not the computed #rrggbb value.

SIRI-924
While not part of the original license text, the heading is required to understand to which software the text refers. See the `/licenses` route.

SIRI-924
Sometimes, neither `sirius` not the method seem to be available. We, therefore, just check for the method directly.

SIRI-924
When using the `color` property, a style sheet may interfere. A default background for spans is more unlikely.

SIRI-924
Checking the previous implementation, we used to always send the hex value. This commit thus restores the previous behaviour. At the same time, though, input of non-RGB colors is no longer possible.

SIRI-924
This makes it more clear to the user which color will actually be sent.

SIRI-924
@jakobvogel jakobvogel mentioned this pull request Feb 13, 2024
2 tasks
@jakobvogel jakobvogel marked this pull request as ready for review February 13, 2024 20:58
@jakobvogel jakobvogel removed the 🖐 Keep open Should not be merged label Feb 13, 2024
This extends commit c58dec1 where the feature was originally introduced. At the request of @ymo-sci, the English keys will provide some first steps towards i18n.

SIRI-924
@jakobvogel jakobvogel merged commit 3374777 into develop Feb 14, 2024
2 checks passed
@jakobvogel jakobvogel deleted the feature/jvo/SIRI-924-Bootstrap-5 branch February 14, 2024 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💣 BREAKING CHANGE Contains non-backwards compatible changes to public methods or changes the behavior of existing code 🤯 Complex PR with extensive changes that should be reviewed with extra care ⬆️ Dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants