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 to current version #10

Closed
wants to merge 2 commits into from

Conversation

diepoe
Copy link
Contributor

@diepoe diepoe commented Oct 2, 2021

This closes #7

🍱 add new screenshots

I made new screenshots on the current MacPass version with macOS BigSur. For the full window screenshots, I sized the window to take 2/3 of my screen width and the full height (13inch MacBook Pro).

💄 update html and css for new screenshots

I just created a seperate directory for the screenshots and updated the paths to the images. I also added some missed attributes on the img elements (alt and loading="lazy").

I didn't got a fullsize searchscreen screenshot, because the search functionality wasn't working correctly on my machine. Maybe you could do one, or we still include the old

screenshot of the webpage with the new screenshots

preview

@mstarke
Copy link
Member

mstarke commented Oct 4, 2021

Awesome. I would love to add those but I think I might have created additional work. The current release looks a lot different for macOS 11 upwards. I also might add the Demo.KDBX file to the repo to make contributions like yours easier.

@diepoe
Copy link
Contributor Author

diepoe commented Oct 4, 2021

Ok, I could recreate them and the demo file would help a lot :) [I installed MacPass with brew, so I have to reinstall it again but that isn't a problem]

@mstarke
Copy link
Member

mstarke commented Feb 10, 2022

@haudraufhaun do you still want to update the screenshots? I did finally push 0.8 so we now have an official version. Adding the new support for TOTP might also be an option.

@diepoe
Copy link
Contributor Author

diepoe commented Feb 13, 2022

I can do it, maybe it takes a week or two because I'm busy with some exams right now 🤓. I'll close this pull request and do it on a totally new & fresh PR :)

@diepoe diepoe closed this Feb 13, 2022
@diepoe diepoe deleted the new-screenshots branch February 13, 2022 16:12
@mstarke
Copy link
Member

mstarke commented Feb 14, 2022

Awesome. Thank you!

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.

Update screenshots to current version
2 participants