Skip to content

Conversation

@dombesz
Copy link
Contributor

@dombesz dombesz commented Jan 8, 2026

Ticket

https://community.openproject.org/wp/70334

What are you trying to accomplish?

Align the project close button in the project dropdown.

Screenshots

Before

image

After

image

What approach did you choose and why?

Move the close button inside the span with the flex spacing.

Merge checklist

  • Added/updated tests
  • Added/updated documentation in Lookbook (patterns, previews, etc)
  • Tested major browsers (Chrome, Firefox, Edge, ...)

Copy link
Contributor

@toy toy left a comment

Choose a reason for hiding this comment

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

It works for portfolios and programs, but sadly broke it for projects (the close button is hidden after the title, same in mobile view):
Image
Image
Image
Image
Image

@dombesz dombesz force-pushed the bug/70334-project-dropdown-active-project-close-button-is-misaligned branch from 3f988c3 to 91892fe Compare January 13, 2026 18:58
@github-actions
Copy link

Caution

The provided work package version does not match the core version

Details:

Please make sure that:

  • The work package version OR your pull request target branch is correct

@dombesz dombesz requested a review from toy January 13, 2026 19:00
@dombesz
Copy link
Contributor Author

dombesz commented Jan 13, 2026

Thanks @toy , it should be fixed now.

Copy link
Contributor

@toy toy left a comment

Choose a reason for hiding this comment

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

Perfect, thank you!

@dombesz dombesz merged commit c73a08d into release/17.0 Jan 14, 2026
16 checks passed
@dombesz dombesz deleted the bug/70334-project-dropdown-active-project-close-button-is-misaligned branch January 14, 2026 13:10
@github-actions github-actions bot locked and limited conversation to collaborators Jan 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants