8000 Revamped about section, new credits section by cryptoAlgorithm · Pull Request #96 · SwiftcordApp/Swiftcord · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Revamped about section, new credits section #96

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

Merged
merged 13 commits into from
Aug 8, 2022
Merged

Conversation

cryptoAlgorithm
Copy link
Member

The about section (in settings) was getting a little crowded and messy with the increasing number of contributors, and manually adding and hardcoding new contributors was getting tiring. Besides, the about section never looked great to start with since it was made in a rush. This PR contains a much-needed facelift and revamp, which moves the credits to its own new section with a much better appearance. Now, contributors are fetched from the GitHub REST API and cached, so they're always up-to-date! My valuable sponsors also get more recognition in the new section.

@cryptoAlgorithm
Copy link
Member Author

The new credits section makes my mouth water xD

Screenshot 2022-08-08 at 9 18 54 PM

Screenshot 2022-08-08 at 9 19 54 PM

@cryptoAlgorithm
Copy link
Member Author

The about section got a new coat of paint too! 🖌️

Screenshot 2022-08-08 at 9 20 23 PM

@cryptoAlgorithm
Copy link
Member Author

Not to mention, all the strings in the new credits section are now in the localisation file, so the community can translate them!

@cryptoAlgorithm cryptoAlgorithm marked this pull request as ready for review August 8, 2022 14:03
@cryptoAlgorithm cryptoAlgorithm merged commit 442a8c1 into main Aug 8, 2022
@cryptoAlgorithm cryptoAlgorithm deleted the new-credits branch August 8, 2022 14:07
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.

1 participant
0