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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃拕 UI - User's page needs improvement #1231

Open
4 tasks
Aibono1225 opened this issue Jan 25, 2024 · 4 comments
Open
4 tasks

馃拕 UI - User's page needs improvement #1231

Aibono1225 opened this issue Jan 25, 2024 · 4 comments
Assignees
Labels
Type: Bug A problem with existing functionality

Comments

@Aibono1225
Copy link
Member

Aibono1225 commented Jan 25, 2024

Based on the email.
From: @adamcogan
Date: Wednesday, January 24, 2024 at 3:34鈥疉M
To: @Aibono1225
Cc: @tiagov8 @JackDevAU @jaydenalchin
Subject: Rules - CSS BUG - 16 hours ago

Describe the Bug

On https://ssw.com.au/rules/user/?author=adamcogan
The last column is cramped and does not have the same font size and icon as the Rules.Widget.

Expected Behavior

The date column needs more space and should be consistent with the Rules.Widget

Tasks

  • Add more space for the date column
  • Ensure the date column and the widget date have the same font size and icon
  • Action to both the 'user's rule' and the 'latest rules.'
  • User's Rule - Change Heading from "Last Modified" to "Last Modified (14)"

image

Thanks!

@Aibono1225 Aibono1225 added the Type: Bug A problem with existing functionality label Jan 25, 2024
@Aibono1225 Aibono1225 self-assigned this Jan 25, 2024
@Aibono1225
Copy link
Member Author

Aibono1225 commented Jan 29, 2024

There are 3 options that we can do with the font size on this page:

Option A: Decrease the column's font size

image

Option B: Increase Widget's font size

image

Option C: Leave the font size as it is

image

@Aibono1225
Copy link
Member Author

Aibono1225 commented Jan 29, 2024

As per my conversation with @adamcogan, @hveraus and @sethdaily, we choose option B.

@tiagov8
Copy link
Member

tiagov8 commented Feb 6, 2024

cc @adamcogan @hveraus @sethdaily

Hi @Aibono1225 ,

Our designers @tiagov8 @jaydenalchin and @bettybondoc have reviewed the options you gave...

  • 2 of us prefered C, and 1 prefered A
  • The 3 of us agreed B to be the last choice

--

I don't think it's a good idea to increase the Widget's font size for the time. I'd like to keep it small to keep the information hierarchy distinct especially for tiles/cards like this.

B feels like cluttered UI


  • 1. Please reconsider this task

  • 2. I think currently we only abbreviate "minutes" as "mins".
    It should be "min" as per https://www.ssw.com.au/rules/right-format-to-write-videos-time-length/
    Please fix

  • 3. We also thought about using abbreviations for the rest of time stamps (5 min ago, 2 hr ago, 4 d ago, 3 wk ago, 3 mo ago, 5 y ago, etc). Not a crucial one because we do have the space for full words, so please check with the Product Owner if this is wanted or not

@tiagov8
Copy link
Member

tiagov8 commented Feb 6, 2024

cc @bettybondoc @jaydenalchin @adamcogan

Hi @Aibono1225

鉂楋笍 IMPORTANT - If you look at "Recently Updated" rules view, the heading still say "ADDED BY" which is inconsistent and confusing.

  • 1. On "Recently Updated" view, that label should change to "UPDATED BY"

  • 2. Someone creates a PR for an update, someone else approves it. Who will be shown as "UPDATED BY"?

  • 3. IMO we should ditch the word "Recently" from the "Sort by" filter. As you know, we can click the other button to reverse the order of rules...showing the oldest created/updated rule... so that label becomes invalid.

  • 4. I also think we should change the other view from "Added" to "Created"

--

Screenshot 2024-02-05 at 4 43 07鈥疨M

Figure: Inconsistent wording leads to confusion

Screenshot 2024-02-05 at 4 53 11鈥疨M

Figure: "Recently Added 15 years ago" is flawed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug A problem with existing functionality
Projects
None yet
Development

No branches or pull requests

2 participants