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

Improve empty state grammar for Assets #6922

Merged
merged 2 commits into from Aug 12, 2020

Conversation

brendo
Copy link
Contributor

@brendo brendo commented Jul 7, 2020

Description of what you did:

This PR is a small tweak to the English translations to improve the empty state for the Media Library plugin.

Before

Screen Shot 2020-07-07 at 10 05 46 am

After

Screen Shot 2020-07-07 at 10 03 27 am

@brendo brendo force-pushed the fix_empty-state-grammar branch 2 times, most recently from 5a311a2 to 4508a7c Compare July 7, 2020 00:10
@codecov
Copy link

codecov bot commented Jul 7, 2020

Codecov Report

Merging #6922 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6922   +/-   ##
=======================================
  Coverage   26.22%   26.22%           
=======================================
  Files        1131     1131           
  Lines       15433    15433           
  Branches     2443     2443           
=======================================
  Hits         4048     4048           
  Misses       9565     9565           
  Partials     1820     1820           
Flag Coverage Δ
#front 18.26% <ø> (ø)
#unit 53.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8040a85...5a50a5a. Read the comment docs.

@soupette
Copy link
Contributor

soupette commented Jul 7, 2020

@maevalienard can you check if the wording is ok for you?

maevalienard
maevalienard previously approved these changes Jul 28, 2020
Copy link

@maevalienard maevalienard left a comment

Choose a reason for hiding this comment

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

LGTM

soupette
soupette previously approved these changes Jul 29, 2020
@soupette soupette added source: core:upload Source is core/upload package issue: enhancement Issue suggesting an enhancement to an existing feature labels Jul 29, 2020
Signed-off-by: Brendan Abbott <brendan.abbott@shopify.com>
Copy link
Contributor

@soupette soupette left a comment

Choose a reason for hiding this comment

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

LGTM

@soupette soupette modified the milestone: 3.1.4 Aug 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue: enhancement Issue suggesting an enhancement to an existing feature source: core:upload Source is core/upload package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants