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-4324: fixed overflow in selected items sidebar #648

Merged
merged 5 commits into from
Nov 30, 2022

Conversation

Gengar-i
Copy link
Contributor

@Gengar-i Gengar-i commented Nov 16, 2022

Question Answer
Tickets IBX-4324
Required https://github.com/ibexa/icons/pull/24
Bug fix? yes
New feature? no
License GPL-2.0

fixed overflow in selected items sidebar, added scroll, mov…

Checklist:

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

@@ -2,7 +2,7 @@
background-color: $ibexa-color-white;
position: fixed;
top: calculateRem(95px);
right: 0;
right: calculateRem(16px);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added this 1 rem to move this sidebar inside content no idea if this was made properly or just forgotten to update

@bogusez
Copy link
Contributor

bogusez commented Nov 23, 2022

@Gengar-i the displayed graphics for selected contents are still wrong.

Copy link
Contributor

@bogusez bogusez left a comment

Choose a reason for hiding this comment

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

Selected contents in sidebar menu have still too big graphics

@Gengar-i Gengar-i force-pushed the IBX-4324-selected-items-sidebar branch from 256867f to 9532c63 Compare November 24, 2022 14:58
@Gengar-i Gengar-i requested a review from bogusez November 29, 2022 10:14
Copy link
Contributor

@bogusez bogusez left a comment

Choose a reason for hiding this comment

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

expand sidebar button is not visible

@sonarqubecloud
Copy link

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

@bogusez
Copy link
Contributor

bogusez commented Nov 30, 2022

Regression tests passed: ibexa/commerce#165

@dew326 dew326 merged commit 5cf6727 into 4.2 Nov 30, 2022
@dew326 dew326 deleted the IBX-4324-selected-items-sidebar branch November 30, 2022 11:56
@Gengar-i
Copy link
Contributor Author

Merged:
4.3: e12546d
main: 8e9c0be

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.

8 participants