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

Fix/ADF-1664/Solar design glitches (part 2) #4033

Merged
merged 6 commits into from
Jun 7, 2024

Conversation

jsconan
Copy link
Contributor

@jsconan jsconan commented Jun 3, 2024

Related to: https://oat-sa.atlassian.net/browse/ADF-1664

Summary

Fix another set of UI glitches in the backoffice when the Solar Design applies and a custom theme is already in place.

Details

A few places were not following well the design:

  • the tab selector when publishing a test (select local or remote)
  • the buttons inside the item form (workflow buttons)
  • the bottom bar for the look & feel (nice to have, but this would be hidden in the end)

The action bars in the lists and remote lists were hidden as they were considered empty.

How to test

  • check out the branch: git checkout -t origin/fix/ADF-1664/solar-design-glitches-part2
  • activate the feature flag: php index.php 'oat\tao\scripts\tools\FeatureFlag\FeatureFlagTool' -s FEATURE_FLAG_SOLAR_DESIGN_ENABLED -v true
  • log in to TAO backoffice
  • look at the pages, compare with the UX mockups

Copy link

github-actions bot commented Jun 3, 2024

Front-end summary Node 18

💯 Total ✅ Passed ⏭️ Skipped ❌ Failed
161 161 0 0

Copy link
Contributor

@shaveko shaveko left a comment

Choose a reason for hiding this comment

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

Looks nice.

The branch name in the PR description's How to test should be updated

Copy link

github-actions bot commented Jun 7, 2024

Version

Target Version 54.14.3
Last version 54.14.2

There are 0 BREAKING CHANGE, 0 feature, 4 fixes

@jsconan jsconan merged commit e220926 into develop Jun 7, 2024
6 checks passed
@jsconan jsconan deleted the fix/ADF-1664/solar-design-glitches-part2 branch June 7, 2024 10:23
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