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

add gallery placeholder #2482

Merged

Conversation

patzick
Copy link
Collaborator

@patzick patzick commented Feb 22, 2019

Related issues

Followup for #2481

Short description and why it's useful

Added placeholder before gallery load.

@pkarw answering to comment - there is placeholder earlier in gallery - it's small image of gallery. It was by default first image from gallery, so we've seen:

  1. Nothing
  2. First image
  3. Slide to specific color
  4. placeholder, which was small first image
  5. Actual image

Now it is:

  1. Placeholder (this PR)
  2. Placeholder of actual image (smaller image so it's blurred)
  3. Actual image

Screenshots of visual changes before/after (if there are any)

aft2

Upgrade Notes and Changelog

  • No upgrade steps required (100% backward compatibility)
  • I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

@patzick patzick requested review from filrak and pkarw February 22, 2019 10:29
@patzick patzick added this to the 1.9 milestone Feb 22, 2019
@patzick patzick merged commit 392604e into vuestorefront:develop Feb 22, 2019
@patzick patzick deleted the feature/product-gallery-placeholder branch February 22, 2019 16:54
@patzick patzick modified the milestones: 1.9, 1.8.3 Feb 26, 2019
@ibugaj
Copy link
Collaborator

ibugaj commented Feb 27, 2019

Looks good :)

@ibugaj ibugaj added the QA approved on branch Testers will add this label after positive check on specific branch. label Feb 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA approved on branch Testers will add this label after positive check on specific branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants