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
Conversation
5a311a2
to
4508a7c
Compare
Codecov Report
@@ 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
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@maevalienard can you check if the wording is ok for you? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
17177fd
to
3bfecc5
Compare
Signed-off-by: Brendan Abbott <brendan.abbott@shopify.com>
3bfecc5
to
de36841
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Brendan Abbott <brendan.abbott@shopify.com> Signed-off-by: Gil Fernandes <gil.fernandes@onepointltd.com>
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
After