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] Fix the "matched runs" sentence color style overriding issue in progress bars #1931

Merged

Conversation

KaroMourad
Copy link
Contributor

@KaroMourad KaroMourad commented Jun 30, 2022

  • Progress bar matched runs color style overrides from main.css chunk in the production build
  • Add success styles on Text component
  • Fix Something went wrong issue in the PopoverContent component

@KaroMourad KaroMourad requested a review from roubkar June 30, 2022 08:23
@KaroMourad KaroMourad self-assigned this Jun 30, 2022
@KaroMourad KaroMourad requested review from rubenaprikyan and arsengit and removed request for roubkar June 30, 2022 08:27
@KaroMourad
Copy link
Contributor Author

KaroMourad commented Jun 30, 2022

Resolves #1932

@KaroMourad KaroMourad force-pushed the fix/progress-bar-matched-runs-color-style branch from 39dac35 to 27f6779 Compare June 30, 2022 11:33
Copy link
Contributor

@rubenaprikyan rubenaprikyan left a comment

Choose a reason for hiding this comment

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

Looks good!

@KaroMourad KaroMourad changed the title [fix] Fix progress bar matched runs color style overriding issue [fix] Fix progress bar matched runs color style overriding issue Jun 30, 2022
@KaroMourad KaroMourad changed the title [fix] Fix progress bar matched runs color style overriding issue [fix] Fix the "matched runs" sentence color style overriding issue in progress bars Jun 30, 2022
CHANGELOG.md Outdated Show resolved Hide resolved
@KaroMourad KaroMourad merged commit bbcc49c into release/3.11.x Jul 1, 2022
@KaroMourad KaroMourad deleted the fix/progress-bar-matched-runs-color-style branch July 1, 2022 16:12
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.

3 participants