-
-
Notifications
You must be signed in to change notification settings - Fork 943
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
✨ Rework the community page to feature more modern resources more clearly #1257
Conversation
Hi @coderanger, thanks for your PR. Good set of changes, happy for it to go in with some tidy ups and tests 🚢 Could you please add a before screenshot as well? |
I was referring to a UX QA, and maybe testing on different mobile devices.
…On Fri, Oct 21, 2022, 12:01 AM Noah Kantrowitz ***@***.***> wrote:
Current desktop screenshot
[image: Screenshot 2022-10-20 at 16-00-31 Django Community Django]
<https://user-images.githubusercontent.com/128243/197074932-b6d5ea7f-a17b-4f37-9269-a5d7595edfff.png>
As for tests, happy to add some but not sure what since there's no logic
changes, this is only copy :)
—
Reply to this email directly, view it on GitHub
<#1257 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAMKKLVI3JF3C3XDVJUD3UTWEHFNHANCNFSM6AAAAAARKSKZUA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Ahh fair :D I posted a mobile screenshot for iOS 12/13 width just using Firefox's dev tools, but will see if I can get my phone to load the dev site directly for a more real test. |
I quite like this, FWIW. 🙂 |
Anything I can do to help move this along? |
This looks great, @coderanger ! Thank you for all your work! 🙏🏼 Can you please rebase your PR agains the current main branch? Please tag me once you're done and I'll do a few tests before merging this in. |
c2d72ab
to
960c7ab
Compare
960c7ab
to
6170945
Compare
539066e
to
3b20b85
Compare
for more information, see https://pre-commit.ci
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.
Looks good to me! Also tested locally 👍🏻
The community page centers older resources like the mailing list and IRC channel, which are no longer the best ways to participate in most parts of our community. You can skip down to the screenshots below but that main change is making a 2x2 grid of the primary calls to action to streamline those, as well as updates to the sidebar to remove deprecated or irrelevant content and generally update things.
Before (desktop view):
After:
Desktop screenshot
Mobile screenshot