Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

IBX-7050: Frontend: top bar and change site switcher #1049

Merged
merged 8 commits into from
Jan 8, 2024
Merged

Conversation

GrabowskiM
Copy link
Contributor

@GrabowskiM GrabowskiM commented Dec 11, 2023

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

Missing content tree part, as there's totally different problem with content tree...

with https://github.com/ibexa/site-context/pull/20

Checklist:

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

@GrabowskiM GrabowskiM marked this pull request as ready for review December 18, 2023 11:26
<div class="ibexa-user-mode-badge">
{{ 'user_mode_badge.focus'|trans|desc('Focus mode') }}
</div>
{% endif %}
Copy link
Contributor

Choose a reason for hiding this comment

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

newline

@@ -0,0 +1,7 @@
{% trans_default_domain 'ibexa_user_mode_badge' %}

{% if ibexa_is_focus_mode_on() == true %}
Copy link
Contributor

Choose a reason for hiding this comment

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

This should also work?

Suggested change
{% if ibexa_is_focus_mode_on() == true %}
{% if ibexa_is_focus_mode_on() %}


&:focus {
border-color: $ibexa-color-white;
box-shadow: 0px 0px 0px calculateRem(4px) rgba($ibexa-color-white, 0.2);
Copy link
Contributor

Choose a reason for hiding this comment

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

nitpick

Suggested change
box-shadow: 0px 0px 0px calculateRem(4px) rgba($ibexa-color-white, 0.2);
box-shadow: 0 0 0 calculateRem(4px) rgba($ibexa-color-white, 0.2);

@GrabowskiM GrabowskiM requested a review from tischsoic December 22, 2023 15:12
Copy link

sonarqubecloud bot commented Jan 2, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
1.2% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

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

QA approved on IbexaDXP 4.6 commerce.

@dew326 dew326 merged commit 9323d21 into main Jan 8, 2024
22 checks passed
@dew326 dew326 deleted the IBX-7050-topbar branch January 8, 2024 08:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants