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

PLANET-7543 Add spinner to Greenpeace Media when searching for images #2314

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mleray
Copy link
Contributor

@mleray mleray commented Jun 25, 2024

Description

See PLANET-7543

Note: the spinner is different from the one in the ticket's screenshot, but this one is probably the newest WP one and it has been approved by the design team.

Testing

You can go to Greenpeace Media either on local or on the jupiter instance. When you search for a term then the spinner should now appear next to the search input on mobile, and next to the Search button in bigger screens.

@mleray mleray self-assigned this Jun 25, 2024
@mleray mleray added the UAT Needed This PR requires User Acceptance Tests before merge label Jun 25, 2024
planet-4 added a commit to greenpeace/planet4-test-jupiter that referenced this pull request Jun 25, 2024
/unhold 0d92a55e-f29b-4905-b93b-94a8c16a5099
@planet-4
Copy link
Contributor

planet-4 commented Jun 25, 2024

Test instance is ready 🚀

🌑 jupiter | admin | blocks report | CircleCI | composer-local.json

⌚ 2024.07.01 11:53:53

@mleray mleray force-pushed the greenpeace-media-search-loading branch from 0d7b9d3 to 34f6ee1 Compare June 25, 2024 09:07
planet-4 added a commit to greenpeace/planet4-test-jupiter that referenced this pull request Jun 25, 2024
/unhold 287f5347-0508-4603-85c2-89d97d51fda5
@mleray mleray force-pushed the greenpeace-media-search-loading branch from 34f6ee1 to 3a87c40 Compare June 25, 2024 09:22
planet-4 added a commit to greenpeace/planet4-test-jupiter that referenced this pull request Jun 25, 2024
/unhold c3f32b09-ce9e-4565-9e12-2d580b88dfa7
@mleray mleray marked this pull request as ready for review June 25, 2024 09:42
@mleray mleray force-pushed the greenpeace-media-search-loading branch from 3a87c40 to ed3f44f Compare July 1, 2024 11:20
planet-4 added a commit to greenpeace/planet4-test-jupiter that referenced this pull request Jul 1, 2024
/unhold a0d8e5ff-54cb-4f75-b3b3-c18e66a6974b
@mleray mleray force-pushed the greenpeace-media-search-loading branch from ed3f44f to c8d3614 Compare July 1, 2024 11:41
This is to indicate to users that their search is being processed
planet-4 added a commit to greenpeace/planet4-test-jupiter that referenced this pull request Jul 1, 2024
/unhold 94993a98-f6c3-4b79-b6e7-f6c9484d6eee
@mleray mleray force-pushed the greenpeace-media-search-loading branch from c8d3614 to 367bcba Compare July 1, 2024 11:44
planet-4 added a commit to greenpeace/planet4-test-jupiter that referenced this pull request Jul 1, 2024
/unhold 42e6e709-28b2-4ff6-92ed-a1503cc14f29
@mleray mleray added the Review label Jul 1, 2024
@mleray mleray requested review from sagarsdeshmukh and removed request for Osong-Michael July 2, 2024 07:34
Copy link
Contributor

@Osong-Michael Osong-Michael left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This component is also used by the Greenpeace Media in the Editor, right now the styles work fine for the Dashboard, but in the Editor it is a bit off, is that something we can fix in this ticket?
Screenshot 2024-07-02 at 10 48 51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review [Test Env] jupiter UAT Needed This PR requires User Acceptance Tests before merge
Projects
None yet
3 participants