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

fix(core): toolbar render spacer before overflow conditionally #12453

Merged
merged 2 commits into from
Oct 8, 2024

Conversation

droshev
Copy link
Contributor

@droshev droshev commented Sep 20, 2024

Related Issue(s)

closes #12239

Description

render spacer before the overflow button conditionally (only if the application doesn't have non-fixed spacer)

@droshev droshev requested a review from a team September 20, 2024 19:28
@droshev droshev self-assigned this Sep 20, 2024
@droshev droshev added this to the Sprint 138 - September 2024 milestone Sep 20, 2024
Copy link

netlify bot commented Sep 20, 2024

Deploy Preview for fundamental-ngx ready!

Name Link
🔨 Latest commit b151598
🔍 Latest deploy log https://app.netlify.com/sites/fundamental-ngx/deploys/670476a294cebd000809eba9
😎 Deploy Preview https://deploy-preview-12453--fundamental-ngx.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

github-actions bot commented Sep 20, 2024

Visit the preview URL for this PR (updated for commit b151598):

https://fundamental-ngx-gh--pr12453-fix-toolbar-spacer-bbhxy52c.web.app

(expires Fri, 11 Oct 2024 00:07:14 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 41b993ee8e451bd7c6770b342ce142dc886eacff

@droshev droshev force-pushed the fix/toolbar-spacer branch 2 times, most recently from 84ff810 to d0d28cd Compare September 23, 2024 23:52
Copy link
Member

@mikerodonnell89 mikerodonnell89 left a comment

Choose a reason for hiding this comment

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

The overflow priority example doesn't look great on small screens:

this pr:
Screenshot 2024-09-24 at 2 04 01 PM

main branch:
Screenshot 2024-09-24 at 2 04 06 PM

Copy link
Member

@mikerodonnell89 mikerodonnell89 left a comment

Choose a reason for hiding this comment

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

Approving as the issue I pointed out was pre-existing, but not apparent in the example due to the example previously not having the spacer. We can address that in another issue

@droshev droshev changed the title fix(core): toolbar render spacer before overflow conditionally [WIP]fix(core): toolbar render spacer before overflow conditionally Oct 1, 2024
@droshev droshev force-pushed the fix/toolbar-spacer branch 2 times, most recently from c0e010d to 30bbfac Compare October 7, 2024 17:33
@droshev droshev force-pushed the fix/toolbar-spacer branch 2 times, most recently from 7274a94 to 68dafb5 Compare October 7, 2024 19:14
@droshev droshev changed the title [WIP]fix(core): toolbar render spacer before overflow conditionally fix(core): toolbar render spacer before overflow conditionally Oct 7, 2024
@droshev
Copy link
Contributor Author

droshev commented Oct 8, 2024

The overflow priority example doesn't look great on small screens:

this pr: Screenshot 2024-09-24 at 2 04 01 PM

main branch: Screenshot 2024-09-24 at 2 04 06 PM

should be fixed now:
Screenshot 2024-10-07 at 20 42 18

@droshev droshev merged commit 6e1dc71 into main Oct 8, 2024
19 checks passed
@droshev droshev deleted the fix/toolbar-spacer branch October 8, 2024 00:43
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.

Bug: Overflow of fdp table toolbar leads to changing spacing when resizing the viewport
4 participants