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

Chapter 8 (Improving UX) revision based on reading session #28

Merged
merged 5 commits into from
Mar 3, 2025

Conversation

GBKS
Copy link
Owner

@GBKS GBKS commented Dec 6, 2024

Another page where we decided in the reading session to do a bigger rewrite. The original content repeated a lot of things already discussed in earlier chapters, and it was (as usual) short on examples. The new focus with this rewrite here is to give more specific examples on how to contribute as a designer, along with examples and reference link.

While working on this, I ended up with a pretty good paragraph on user groups. That was not a good fit for this page in the end, so I moved it to the design process page.

I don't think this initial commit is super good, but I'd like to get it out there for feedback. Please let me know what you think.

🤿Check the preview🎣

Another page where we decided in the reading session to do a bigger rewrite. The original content repeated a lot of things already discussed in earlier chapters, and it was (as usual) short on examples. The new focus with this rewrite here is to give more specific examples on how to contribute as a designer, along with examples and reference link.

While working on this, I ended up with a pretty good paragraph on user groups. That was not a good fit for this page in the end, so I moved it to the design process page.

I don't think this initial commit is super good, but I'd like to get it out there for feedback. Please let me know what you think.
@GBKS GBKS added the enhancement New feature or request label Dec 6, 2024
@GBKS GBKS self-assigned this Dec 6, 2024
Copy link

netlify bot commented Dec 6, 2024

Deploy Preview for opendesign-guide ready!

Name Link
🔨 Latest commit fa56e06
🔍 Latest deploy log https://app.netlify.com/sites/opendesign-guide/deploys/67c57d45b8f5c40008cf2415
😎 Deploy Preview https://deploy-preview-28--opendesign-guide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@Erioldoesdesign Erioldoesdesign 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: love the examples here and the mini user scenario/statement re. Maria

@GBKS
Copy link
Owner Author

GBKS commented Jan 7, 2025

@paperpsych I'd appreciate your review on this update as well. Your previous one on chapter 7 was super helpful.

@GBKS GBKS changed the title Chapter 8 revision based on reading session Chapter 8 (Improving UX) revision based on reading session Jan 13, 2025
@GBKS GBKS requested a review from Copilot February 27, 2025 09:24

Choose a reason for hiding this comment

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

PR Overview

This pull request revises Chapter 8 to improve user experience guidance by providing clearer examples and more specific advice, while also relocating content about user groups to better fit the design process narrative.

  • Revised Chapter 8 text to include concrete examples and analogies that illustrate UX improvements.
  • Restructured section headings and content to emphasize the importance of understanding the community and usability.
  • Expanded the user research section in the design process page to include additional user groups and an illustrative persona.

Reviewed Changes

File Description
content/8-improving-ux.md Updated narrative with detailed examples and restructured sections to focus on pragmatic UX improvements.
content/5-design-process.md Expanded the user research section by adding an extra user group and detailed persona example.

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Copy link
Contributor

@paperpsych paperpsych left a comment

Choose a reason for hiding this comment

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

Yello! Didn't change the crux and essence of the sections, only reframed sentences for better flow. All of the "For Example" pieces are written by AI so please cross-check once if they're accurate, rest looks in good shape.

I have some more formatting to do but that can be done later with a PR. It's GTG from my end.

If there's any feedback about the review, please let me know. Thank you !!

@GBKS
Copy link
Owner Author

GBKS commented Mar 3, 2025

Thank you for the reviews, @Erioldoesdesign and @paperpsych. Going to merge this one.

And that will conclude the feedback implementation from our reading sessions, making this guide a ton more informative and useful. Huge thanks to all who chipped in.

@GBKS GBKS merged commit 7bf28ee into main Mar 3, 2025
4 checks passed
@GBKS GBKS deleted the feature/chapter-8-revision branch March 3, 2025 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants