WordPress.org

Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #37661, comment 68


Ignore:
Timestamp:
10/03/2016 07:22:45 PM (3 years ago)
Author:
karmatosed
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #37661, comment 68

    initial v1  
    11I've just done a review and would like to pass on some feedback as I walked through. The good news is that during the review, the flow more or less works. For this purpose, I used only on desktop. I will do device testing during this week.
    22
    3 [[Image(https://cldup.com/oVd0JvZmKq.png, 100%)]]
     3[[Image(https://cldup.com/oVd0JvZmKq.png, 50%)]]
    44
    55- I wonder if in the above you should be able to click the search icon? It feels weird to just be there unable to interact.
    66
    7 [[Image(https://cldup.com/NVU2mUjz6w.png, 100%)]]
     7[[Image(https://cldup.com/NVU2mUjz6w.png, 50%)]]
    88
    99- I found it weird to not be able to close the open filtering. It would be great to maybe consider adding this interaction.
    1010
    11 [[Image(https://cldup.com/VY5lJM_HNs.png, 100%)]]
     11[[Image(https://cldup.com/VY5lJM_HNs.png, 50%)]]
    1212
    1313- This section feels like there are some formatting issues with the background. Maybe add some spacing too?
     
    1515- I'm not sure about the checkmark for installed. It feels weird to get checks for that and then featured. I'm a little confused as the check seems diluted as a result. Maybe we can see this more in testing. Just noting as a possible pain point.
    1616
    17 [[Image(https://cldup.com/BL5bFd2B7x.png, 100%)]]
     17[[Image(https://cldup.com/BL5bFd2B7x.png, 50%)]]
    1818
    1919- It doesn't seem to show the preview theme name when you click it, see above.
     
    2525- On using both flows (wp-admin and new one), I begun to think if we need the blue checkmark or not? I sort of feel it works better without.
    2626
    27 [[Image(https://cldup.com/gHzS-Daiaq.png, 100%)]]
     27[[Image(https://cldup.com/gHzS-Daiaq.png, 50%)]]
    2828
    2929- It seems weird to get loading for both of these above.
     
    3131- The filtering feels super clumsy, I wonder if there is anyway we can do sub-sectioning open/close? We end up with these giant scrolling sections like this:
    3232
    33 [[Image(https://cldup.com/NI-2DrP4Mz.png, 100%)]]
     33[[Image(https://cldup.com/NI-2DrP4Mz.png, 50%)]]
    3434
    3535- It seems a little odd that you can click the 'installed'. It's only one and clicking doesn't do anything.
    3636
    37 [[Image(https://cldup.com/bj06E-Ud1F.png, 100%)]]
     37[[Image(https://cldup.com/bj06E-Ud1F.png, 50%)]]