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 A Trail of Footprints #185

Merged
merged 3 commits into from
May 16, 2024
Merged

Add A Trail of Footprints #185

merged 3 commits into from
May 16, 2024

Conversation

danirabbit
Copy link
Member

  • I've read and followed the general guidelines
    • Looks good in Pantheon on elementary OS
    • Not too distracting if a non-maximized window is open
    • No text or logos
    • No people
    • At least 3200×1800 px
  • I've updated debian/copyright with:
    • The name of the wallpaper and/or its original author
    • A link to where it can be downloaded
    • Included under the respective license section (or created a new one if appropriate)
  • I've added the artist metadata using exiftool

Why it should be included:

It's quite different from our other wallpapers

Screenshot(s) of it in Pantheon on elementary OS:

Screenshot from 2024-05-15 17 24 37

https://unsplash.com/photos/a-trail-of-footprints-in-the-sand-of-a-beach-A9mr3TPoj0k

@danirabbit danirabbit requested a review from a team May 16, 2024 00:25
micahilbery
micahilbery previously approved these changes May 16, 2024
Copy link
Member

@micahilbery micahilbery left a comment

Choose a reason for hiding this comment

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

I like the neutral tans of the sand. If the little jet line in the sky on the right could be edited out that would be nice, but I won't hold it up for that.

@danirabbit
Copy link
Member Author

@micahilbery I got you fam. Good eye!

@danirabbit danirabbit requested a review from micahilbery May 16, 2024 18:21
Copy link
Member

@micahilbery micahilbery left a comment

Choose a reason for hiding this comment

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

LGTM! :shipit:

@danirabbit danirabbit merged commit d6350f6 into master May 16, 2024
3 checks passed
@danirabbit danirabbit deleted the danirabbit/david-emrich branch May 16, 2024 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants