- Issue created by @chrisfromredfin
- Merge request !568Issue #3499417: Project Browser CSS minor fix β (Merged) created by Unnamed author
Please review the changes. I have attached before and after Screenshots for reference.
- π¨πSwitzerland saschaeggi Zurich
The search icon seems misaligned and not the remove button
I have added some padding in the search icon. Can you verify the alignment once.
- πΊπΈUnited States mcgovernm North Carolina
Looks like the CSS change was missing from the scss file, build is now passing. Marking as needs review.
- First commit to issue fork.
- π¨π¦Canada zetagraph
Reviewed Search icon overlap fix and things look good.
While in there, I've made a small tweak to the "security" icons, as they were not consistent in sizing.
HI @zetagraph,
Thank you for identifying the issue with the size of the security iconsβI appreciate your attention to detail. However, I believe this might not be directly related to the current issue. I belive we could keep this issue only for fixing the overlap issue of search icon and cross icon. Perhaps we could create a separate issue specifically for this?Please feel free to share your perspective and correct me if Iβm mistaken as i am very new to contribution.
Increase the inset-inline-end more on `#project-browser .search__search-clear` as it was still overlapping with search icon when it gets focus.
tomislav.matokovic β made their first commit to this issueβs fork.
I have successfully replicated this issue and tested the solution.
-
saschaeggi β
committed 832960ec on 4.0.x authored by
sandip poddar β
Issue #3499417: Project Browser CSS minor fix
-
saschaeggi β
committed 832960ec on 4.0.x authored by
sandip poddar β
- π¨πSwitzerland saschaeggi Zurich
Thanks for working on this, I've pushed some more fixes and made some slight adjustments but overall it looked great ππ€
- π¨πSwitzerland saschaeggi Zurich
Included in the 4.0.4 release: https://www.drupal.org/project/gin/releases/4.0.4 β
Automatically closed - issue fixed for 2 weeks with no activity.