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

priority-overflow should be able to handle gaps and regular margins #23066

Open
ling1726 opened this issue May 18, 2022 · 5 comments
Open

priority-overflow should be able to handle gaps and regular margins #23066

ling1726 opened this issue May 18, 2022 · 5 comments

Comments

@ling1726
Copy link
Member

Currently the overflow manager has a padding property to help account for space that is not a part of element content boxes (mainly margin).

However, in the case where regular margins are set on overflow items, or the container uses a gap, it would be useful to take that into account during available space calculation.

  • getComputedStyles to actually read the computed margin ?
  • some kind of regularPadding prop that applies to all overflow items equally
  • each item has a padding prop ?
@msft-fluent-ui-bot
Copy link
Collaborator

Because this issue has not had activity for over 150 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

@msft-fluent-ui-bot msft-fluent-ui-bot added the Resolution: Soft Close Soft closing inactive issues over a certain period label Jun 22, 2023
@ling1726 ling1726 reopened this Jun 26, 2023

Because this issue has not had activity for over 150 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

Because this issue has not had activity for over 150 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

@evlevy
Copy link
Contributor

evlevy commented Sep 20, 2024

+1 this is highly annoying visual defect for (I assume) a very common use case.

https://stackblitz.com/edit/qiaafd?file=src%2Fexample.tsx

@gouttierre gouttierre added Needs: Backlog review Partner Ask and removed Resolution: Soft Close Soft closing inactive issues over a certain period labels Sep 23, 2024
@gouttierre
Copy link
Contributor

gouttierre commented Sep 23, 2024

📢

  • This Item is affecting partners, and needs to be discussed for triage.
  • Moving to backlog review for next steps.
  • Updated the priority to high

cc~ @evlevy , @tudorpopams , @jurokapsiar , @JustSlone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants