guardian / grid

The Guardian’s image management system
https://www.theguardian.com/info/developer-blog/2015/aug/12/open-sourcing-grid-image-service
Apache License 2.0
1.44k stars 120 forks source link

IMAGEDAM-1484 - Add label 'Collections' to icon in Navigation #4140

Closed Conalb97 closed 11 months ago

Conalb97 commented 12 months ago

What does this change?

To raise visibility and awareness of Collections, the word 'Collections' has been added next to the Collections icon in the navigation. The behaviour of this label must match the existing labels and as such is resizeable, so the text is removed when the user shrinks the page.

image image

How should a reviewer test this change?

These changes can be tested by confirming that when landing on BBC Images tester can clearly see the link to Collections in the navigation, in the same style as 'Show preview' and 'Show info panel' buttons. Clicking on the icon or the word 'Collections' opens up the Collections panel.

On smaller breakpoints where there isn't room for the full button just the icon should display - consistent with 'Show preview' and 'Show info panel' buttons which just display as icons.

Who should look at this?

@guardian/digital-cms

Tested? Documented?

paperboyo commented 12 months ago

Hi, Thanks for that. Eminently sensible, I think.

One tiny thing: I think we should now remove the duplicated/redundant now second instance of “Collections” (highlighted below):

image

abdelrahmansd commented 12 months ago

Hi, Thanks for that. Eminently sensible, I think.

One tiny thing: I think we should now remove the duplicated/redundant now second instance of “Collections” (highlighted below):

image

Thanks Mat for the feedback, will update the PR now

prout-bot commented 11 months ago

Seen on auth, usage, image-loader, metadata-editor, thrall, leases, cropper, media-api, kahuna (created by @Conalb97 and merged by @twrichards 10 minutes and 53 seconds ago) Please check your changes!

prout-bot commented 11 months ago

Seen on collections (created by @Conalb97 and merged by @twrichards 11 minutes ago) Please check your changes!