**Read through the team’s feedback and decide with @JTannerWMF on how to move forward with it:####**To do list based on the design team’s feedback ([[ https://www.figma.com/file/XuMaiGvAgrJKvsm2esbm75/Edit-history-T297759?node-id=1%3A6 | Figma ]])**
- JK:[ ] CM: One note around the orange border – the goal was to connect the changes to the later revision (eg. who made the changes)
- [ ] BM: Some colors are not contrasted enough so they are not accessible.
- I know you are just joking, but seriously your design is not a “ripoff,” we are building a design system right now because it’s important for us to have consistency across components (+ ∞ CLM)- [ ] BM: Green color is too light. Also, when green is on yellow box is even less contrasted.
- Feedback for Barbara/Kieran - it would be good to see how the colors are used across the board for platforms doing rev. History- [ ] BM: Yellow is not contrasted enough when you use it as text.
- AA - [ ] RH: +1 - think this is also when Yellow30 is on Yellow90 instead of white, perhaps something to consider for future Codex colour palette improvements?
- From/To could be a row layout instead of a column layout to potentially display more text or we could save a line of text by removing the labels “From/To” and use a visual to represent direction,- [ ] BM: What is the difference between using blue or yellow colors in boxes? There are too many colors in the same screen (green, blue, yellow). If we use too mant colors may be we should explain for what is each color to avoid the user getting lost. for example instead of a vertical separator a separatorMay be we need tags that resemble the “>”,say NEW or OLD. I think+1 (KM – I had there is a similar component in section translation too. same question over the need for the blue/yellow colour difference)
- Wondering if the chart needs scales :)- [ ] RH: Large title text - how does this truncate or wrap?
- Card layout vs row layout, we could increase the information density if we opt for the latter.- [ ] RH: Is the spark chart meant to be interactive? If not then wonder if it needs to take up so much room.
- [ ] RH: Also wondering if it is meant to showing rev changes bar rather than line chart? The overall layout is quite airy while I assume that someone who looks at revision comparison needs to see “many” things at once.Could be a cool change in future to enable tapping to enable interactive Revision slider view (also consistent with web)
- CLM:- [ ] RH: Change the orange of the check icon to same Yellow30
- Really like the accessibility updates that you made!
- Cleaning up the comparison view looks great too!
- One note around the orange border – the goal was to connect the changes to the later revision (eg. who made the changes)
- PG:
- Great improvement. I think it is a good sign that what is well resolved in a platform can be transfer to others.
- Selecting more than two revisions shows a toast message to communicate it is not possible. I wonder if this could be communicated by the way the selectors work. For example, tapping on a third one could update the previous selection or the closest one (thinking along the lines of adjusting a range, like when selecting text)
- Idea for future: To make diffs more understandable I wonder if it is possible to provide a kind of “instant preview” that allow to visually understand a change. (+1 AV)
- BM:
- Some colors are not contrasted enough so they are not accessible.
- Green color is too light. Also, when green is on yellow box is even less contrasted.
- Yellow is not contrasted enough when you use it as text.
- RH: +1 - think this is also when Yellow30 is on Yellow90 instead of white, perhaps something to consider for future Codex colour palette improvements?
- What is the difference between using blue or yellow colors in boxes? There are too many colors in the same screen (green, blue, yellow). If we use too mant colors may be we should explain for what is each color to avoid the user getting lost. May be we need tags that say NEW or OLD. +1 (KM – I had the same question over the need for the blue/yellow colour difference)
- PG: If I recall correctly, historically diffs were using red for deletion and green for additions, and the community updated that to orange/yellow and blue instead due to concerts about color blindness.
- AV:
- I didn’t get to see the history/progress of this, but I really like what I see!
- When the mode changes from view mode to ‘compare mode’, what if the background changed to a light gray, to further reinforce the mode has changed?
- RH:
- Large title text - how does this truncate or wrap?
- Vertical alignment consistency - should the icon/button always be vertically centered or top with sibling text elements? For example the Revision History + Title text and the “Watching” label, selected revision radio and time text.
- Is the spark chart meant to be interactive? If not then wonder if it needs to take up so much room. Also wondering if it is meant to showing rev changes bar rather than line chart? Could be a cool change in future to enable tapping to enable interactive Revision slider view (also consistent with web)
- Change the orange of the check icon to same Yellow30
- SP:
- Love the search and sticky header with filters
- It would be worthwhile to re-evaluate what we put on the screen and in what format, where we want to draw the user's eyes to. The designs are already very clean, now it’s just a matter of trying to further push it by redesigning certain elements and how much visual importance it get to further reduce visual elements on the screen.
- Noticed that anon edit means anonymous edit. Is it a commonly used short version? I wouldn’t have known if not mentioned in the meeting.
- KM:
- Small design system point: Does the bright yellow highlight colour need to be added to the design system? Or is that OS native?- [ ] SP:Noticed that anon edit means anonymous edit. Is it a commonly used short version? I wouldn’t have known if not mentioned in the meeting.