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

IBX-4328: Hover on icons preview switcher #656

Merged
merged 3 commits into from
Jan 10, 2023

Conversation

GrabowskiM
Copy link
Contributor

Question Answer
Tickets https://issues.ibexa.co/browse/IBX-4328
Bug fix? yes
New feature? no
BC breaks? no
Tests pass? yes
Doc needed? no
License GPL-2.0

Checklist:

  • Coding standards ($ composer fix-cs)
  • Ready for Code Review

@GrabowskiM GrabowskiM changed the base branch from 4.2 to 4.3 December 1, 2022 09:43
@GrabowskiM GrabowskiM force-pushed the IBX-4328-preview-icons-switcher-hover branch from 1f6c5a5 to 28c1b9a Compare December 1, 2022 11:10
Copy link
Contributor

@katarzynazawada katarzynazawada left a comment

Choose a reason for hiding this comment

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

In Page Builder hover on icon preview still has wrong color, see https://recordit.co/MmVbNljGNx

Copy link
Contributor

@katarzynazawada katarzynazawada left a comment

Choose a reason for hiding this comment

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

There is a difference between hover on selected icon in PB and admin-ui
https://recordit.co/en6Nu5oyd1
https://recordit.co/LIDd8MtPFJ

@sonarcloud
Copy link

sonarcloud bot commented Dec 20, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@dew326 dew326 merged commit 7d316a1 into 4.3 Jan 10, 2023
@dew326 dew326 deleted the IBX-4328-preview-icons-switcher-hover branch January 10, 2023 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants