Small UX fixes closes #281 #293

Open
carla wants to merge 4 commits from feature/281_uxfixes into main
Owner

Description of the implemented changes

The changes were:

  • Bugfixing
  • New Feature
  • Breaking Change
  • Refactoring

What has been changed?

  • payment translation changed
  • new member field translation fixes for german
  • copy mail adresses buttons not hiding but disabled
  • update club name on save implemented

Definition of Done

Code Quality

  • No new technical depths
  • Linting passed
  • Documentation is added were needed

Accessibility

  • New elements are properly defined with html-tags
  • Colour contrast follows WCAG criteria
  • Aria labels are added when needed
  • Everything is accessible by keyboard
  • Tab-Order is comprehensible
  • All interactive elements have a visible focus

Testing

  • Tests for new code are written
  • All tests pass
  • axe-core dev tools show no critical or major issues

Additional Notes

## Description of the implemented changes The changes were: - [X] Bugfixing - [X] New Feature - [ ] Breaking Change - [ ] Refactoring ## What has been changed? - payment translation changed - new member field translation fixes for german - copy mail adresses buttons not hiding but disabled - update club name on save implemented ## Definition of Done ### Code Quality - [x] No new technical depths - [x] Linting passed - [x] Documentation is added were needed ### Accessibility - [ ] New elements are properly defined with html-tags - [ ] Colour contrast follows WCAG criteria - [ ] Aria labels are added when needed - [ ] Everything is accessible by keyboard - [ ] Tab-Order is comprehensible - [ ] All interactive elements have a visible focus ### Testing - [ ] Tests for new code are written - [x] All tests pass - [x] axe-core dev tools show no critical or major issues ## Additional Notes <!--- Add any additional information for the reviewers here -->
carla added 4 commits 2025-12-15 08:54:51 +01:00
carla changed title from #281Small UX fixes closes to Small UX fixes closes #281 2025-12-15 09:06:31 +01:00
All checks were successful
continuous-integration/drone/push Build is passing
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/281_uxfixes:feature/281_uxfixes
git checkout feature/281_uxfixes

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout main
git merge --no-ff feature/281_uxfixes
git checkout feature/281_uxfixes
git rebase main
git checkout main
git merge --ff-only feature/281_uxfixes
git checkout feature/281_uxfixes
git rebase main
git checkout main
git merge --no-ff feature/281_uxfixes
git checkout main
git merge --squash feature/281_uxfixes
git checkout main
git merge --ff-only feature/281_uxfixes
git checkout main
git merge feature/281_uxfixes
git push origin main
Sign in to join this conversation.
No description provided.