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(css): Fix margin-bottom for list items #4596

Merged
merged 2 commits into from Jul 28, 2023
Merged

Conversation

mejo-
Copy link
Member

@mejo- mejo- commented Jul 28, 2023

Fixes: #4547

Required after 774a42b.

🏚️ Before 🏑 After
Bildschirmfoto vom 2023-07-28 15-56-06 Bildschirmfoto vom 2023-07-28 15-57-01

@cypress
Copy link

cypress bot commented Jul 28, 2023

1 flaky tests on run #11384 β†—οΈŽ

0 149 2 0 Flakiness 1

Details:

fix(css): Fix margin-bottom for list items
Project: Text Commit: 0cfd1dac6e
Status: Passed Duration: 04:47 πŸ’‘
Started: Jul 28, 2023 2:35 PM Ended: Jul 28, 2023 2:39 PM
FlakinessΒ  cypress/e2e/api/SyncServiceProvider.spec.js β€’ 1 flaky test

View Output Video

Test Artifacts
Sync service provider > recovers from a dropped message Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

Copy link
Collaborator

@max-nextcloud max-nextcloud left a comment

Choose a reason for hiding this comment

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

Code looks good. Maybe design review would make sense?

@jancborchardt @nimishavijay Could you check the screenshots in the description?

@mejo-
Copy link
Member Author

mejo- commented Jul 28, 2023

@jancborchardt @nimishavijay Could you check the screenshots in the description?

Given that it just fixes a regression, I'm sure it's the correct fix πŸ˜‡

@mejo-
Copy link
Member Author

mejo- commented Jul 28, 2023

/compile

@nimishavijay
Copy link
Member

Looks good! :)

mejo- and others added 2 commits July 28, 2023 16:22
Fixes: #4547

Required after 774a42b.

Signed-off-by: Jonas <jonas@freesources.org>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@mejo- mejo- force-pushed the fix/list_item_margin_bottom branch from 227e4b9 to 0cfd1da Compare July 28, 2023 14:28
@mejo- mejo- merged commit 129019a into main Jul 28, 2023
31 checks passed
@mejo- mejo- deleted the fix/list_item_margin_bottom branch July 28, 2023 14:48
@mejo-
Copy link
Member Author

mejo- commented Jul 28, 2023

/backport dee7cea to stable27

@mejo-
Copy link
Member Author

mejo- commented Jul 28, 2023

/backport dee7cea to stable26

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.

[UI] list items are too separated in a list
4 participants