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

Update screenshots #47

Closed
helen opened this issue Nov 26, 2019 · 0 comments · Fixed by #58
Closed

Update screenshots #47

helen opened this issue Nov 26, 2019 · 0 comments · Fixed by #58
Assignees
Labels
needs:documentation This requires documentation. type:good-first-issue Good for newcomers.
Milestone

Comments

@helen
Copy link
Contributor

helen commented Nov 26, 2019

Is your enhancement related to a problem? Please describe.

The current screenshot is now slightly outdated because of the addition of the revisions link.

Describe the solution you'd like

I'd like to update the current screenshot (contents can be found in CONTRIBUTING.md) and also add screenshots of the revisions screen and the new notice that shows if you have an ads.txt file on the server. These should be at HiDPI (Retina).

Designs
See current

Describe alternatives you've considered
n/a

Additional context
This can be done independently of a milestone - if other PRs are merged before this is handled, I suggest making a PR against master instead of develop, then once that PR is merged, allow the readme/asset updater Action to run before merging master back into develop.

@jeffpaul jeffpaul added the needs:documentation This requires documentation. label Nov 26, 2019
@jeffpaul jeffpaul added this to the 1.3.0 milestone Nov 27, 2019
@helen helen modified the milestones: 1.3.0, 1.2.1 Mar 20, 2020
@jeffpaul jeffpaul self-assigned this Apr 10, 2020
@jeffpaul jeffpaul mentioned this issue Apr 10, 2020
6 tasks
@helen helen closed this as completed in #58 May 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs:documentation This requires documentation. type:good-first-issue Good for newcomers.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants