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

💄UI - Improve "Latest Rules" section on sidebar #369

Closed
tiagov8 opened this issue Jan 27, 2022 · 3 comments
Closed

💄UI - Improve "Latest Rules" section on sidebar #369

tiagov8 opened this issue Jan 27, 2022 · 3 comments

Comments

@tiagov8
Copy link
Member

tiagov8 commented Jan 27, 2022

cc: @Geordie88


Hi team,

The Latest Rules section on the sidebar of a profile in SSW.People (e.g.: https://www.ssw.com.au/people/bryden-oliver) looks heavy and cluttered...

Screen Shot 2022-01-27 at 5 53 13 PM

... the same "widget" on Rules homepage looks way cleaner:

Screen Shot 2022-01-27 at 5 53 08 PM

  1. Please apply the design from SSW.Rules on SSW.People
@tiagov8
Copy link
Member Author

tiagov8 commented Mar 6, 2023

cc @Geordie88

Hey @bradystroud

Looks like this was actioned... but still Issue is still Open here... Can you check who fixed it?

Screenshot 2023-03-06 at 12 30 26 PM

@tiagov8
Copy link
Member Author

tiagov8 commented Mar 6, 2023

cc @adamcogan

⚠️ UPDATE FOR BEFORE CLOSING THE ISSUE

Hey @bradystroud

Based on a feedback from @amankumarrr

I think the term "Latest Rules" may not be accurate since the list shown here is for rules edited/created by the person profile. This is not clear.

  1. Can we change from

    LATEST RULES

    to

    MY RULES
    or
    RULES CONTRIBUTIONS

  2. Are we able to hide the whole section is no rule has been edited by the employee? E.g. Aman's profile, which is empty

  3. I would also have only 6 instead of 10 to avoid scrolling with no content. Example:

Screenshot 2023-03-06 at 12 38 38 PM

  1. Close the Issue after everything is done and live ;)

@bradystroud
Copy link
Member

bradystroud commented Mar 7, 2023

cc @BrookJeynes @amankumarrr
Hey @tiagov8

Thanks for the feedback 🙂
I created issues to cover all these tasks

Closing this one as the original issue is fixed

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

No branches or pull requests

2 participants