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

[53809] Update PageHeaders & SubHeaders in the (rails) global index pages (fix issues) #16882

Conversation

bsatarnejad
Copy link
Contributor

@bsatarnejad bsatarnejad commented Oct 2, 2024

Ticket

https://community.openproject.org/projects/design-system/work_packages/53809/activity

What are you trying to accomplish?

Fix remaining issues from previous PR: Breadcrumbs correction in Home, Meetings, Time and costs pages, use Primer button for adding a new board, calendar, team planner.

Screenshots

Before:
Screenshot 2024-10-02 at 10 26 21

After:
Screenshot 2024-10-02 at 10 26 45

Before:
Screenshot 2024-10-02 at 10 25 24

After:
Screenshot 2024-10-02 at 10 25 36

Before:
Screenshot 2024-10-02 at 10 24 53

After:
Screenshot 2024-10-02 at 10 25 05

Before:
Screenshot 2024-10-02 at 10 22 52

After:
Screenshot 2024-10-02 at 10 23 30

What approach did you choose and why?

Use Primer button for creating a new board, calendar, team planner and fix breadcrumbs.

Merge checklist

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

@bsatarnejad bsatarnejad requested a review from HDinger October 2, 2024 12:16
@bsatarnejad bsatarnejad self-assigned this Oct 2, 2024
Copy link
Contributor

@HDinger HDinger left a comment

Choose a reason for hiding this comment

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

Hi @bsatarnejad

  • Please restore the addButtonComponent and use a Primer Button inside. Currently you have a lot of duplicated code, see my comment in code.

Further, I have some minor remarks:

  • On the project list, the name of the Organisation is missing which is incosnistent to the rest of the application
Bildschirmfoto 2024-10-04 um 08 51 17
  • On the notification center, it still says "Home" instead of the organisation name
Bildschirmfoto 2024-10-04 um 09 00 06
  • In all affected project pages, the breadcrumb now says "Organisation name / Project name / module name" which is inconsistent to the pages that we already replaced which only say "Project name / module name"
Bildschirmfoto 2024-10-04 um 08 53 37 Bildschirmfoto 2024-10-04 um 09 02 00
  • I think the "Export" button in the Time&Costs module should be part of the PageHeader and not the SubHeader. Please double check with the product team.
Bildschirmfoto 2024-10-04 um 08 52 29
  • In the meetings module, the header is not reflecting the selection in the sidemenu
Bildschirmfoto 2024-10-04 um 08 51 42

Copy link
Contributor

Choose a reason for hiding this comment

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

Why didn't you just change this component to use a Primer Button? Now, you duplicated a lot of code in the individual modules. The SubHeader can deal with custom components on the right side, so that should not be a problem.

@HDinger
Copy link
Contributor

HDinger commented Oct 11, 2024

I cherry-picked most of the commits to #16934 except for those that affect the AddButtonComponent. I think we misunderstood each other here. 🙈 I changed it in the other PR, maybe we can have a chat about what I had in mind.

@HDinger HDinger closed this Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants