Skip to content

Comments

fix alignment issue with deals notification dot#155

Merged
eebbesen merged 2 commits intomainfrom
notification_fix_deals
Jun 27, 2025
Merged

fix alignment issue with deals notification dot#155
eebbesen merged 2 commits intomainfrom
notification_fix_deals

Conversation

@eebbesen
Copy link
Owner

No description provided.

@eebbesen eebbesen requested a review from Copilot June 27, 2025 21:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adjusts the horizontal positioning of the notification dot on the gift cards navigation link.

  • Updated the utility class from start-100 to start-40 on the <span> in the layout
  • Updated the corresponding test to expect the new class name

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
MainLayout.razor Adjusted the CSS utility class for the notification dot
MainLayoutTest.razor Updated the test assertion to match the new utility class
Comments suppressed due to low confidence (1)

StpFoodBlazor/Components/Layout/MainLayout.razor:26

  • Ensure that start-40 is a valid Bootstrap utility class; if it's not, consider defining a custom CSS class or use inline styles to achieve the 40% offset.
                            <span id="giftcards-exist" class="position-absolute top-0 start-40 translate-middle p-1 bg-danger border border-light rounded-circle"></span>

@eebbesen eebbesen merged commit f73c8b0 into main Jun 27, 2025
2 of 3 checks passed
@eebbesen eebbesen deleted the notification_fix_deals branch June 27, 2025 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant