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

EZP-29650: Fix Selection field (Content Type fields) arrow color #664

Merged

Conversation

tischsoic
Copy link
Contributor

Question Answer
Tickets https://jira.ez.no/browse/EZP-29650
Bug fix? yes
New feature? no
BC breaks? no
Tests pass? yes
Doc needed? no
License GPL-2.0

Fixes 1. problem - pointer cursor.
2. problem was fixed in: #648 by the by of other task.
Previous PR to this task was closed: #644.
More info in JIRA.

Checklist:

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

@@ -3,7 +3,7 @@
position: relative;
color: $ez-color-base-dark;
font-weight: 700;
cursor: pointer;
cursor: initial;
Copy link
Member

Choose a reason for hiding this comment

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

Do we need initial? Cannot we just remove it?

Copy link
Contributor

Choose a reason for hiding this comment

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

If you don't need it, then remove it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It overrides: .ez-field-edit--ezselection .ez-data-source which has cursor: pointer;.

Copy link
Contributor

Choose a reason for hiding this comment

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

So, maybe you should fix it there, hm?

Copy link
Contributor Author

@tischsoic tischsoic Oct 3, 2018

Choose a reason for hiding this comment

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

I didn't want to break something.

Copy link
Contributor

Choose a reason for hiding this comment

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

ez-custom-dropdown comes from ezselection fieldtype. You can remove it safely.

@@ -3,7 +3,7 @@
position: relative;
color: $ez-color-base-dark;
font-weight: 700;
cursor: pointer;
cursor: initial;
Copy link
Contributor

Choose a reason for hiding this comment

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

If you don't need it, then remove it.

@barbaragr barbaragr self-assigned this Oct 3, 2018
Copy link

@barbaragr barbaragr left a comment

Choose a reason for hiding this comment

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

Works fine now :)

@tischsoic tischsoic force-pushed the EZP-29650-content-type-fields-selection-field-UI-issues branch from e260774 to b02fa7c Compare October 3, 2018 14:20
@tischsoic
Copy link
Contributor Author

ping @sunpietro

@lserwatka lserwatka merged commit f979569 into master Oct 4, 2018
@lserwatka lserwatka deleted the EZP-29650-content-type-fields-selection-field-UI-issues branch October 4, 2018 09:12
ViniTou pushed a commit that referenced this pull request May 12, 2023
* Added getElement method

* Added getCell method

Co-authored-by: katarzynazawada <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
5 participants