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

reorganize PT page #72

Closed
wants to merge 2 commits into from
Closed

reorganize PT page #72

wants to merge 2 commits into from

Conversation

mfsilva22
Copy link
Contributor

I think this organization has a better flow (introduce concepts first, then use them in following text).

Copy link

vercel bot commented Mar 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
marketing ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 15, 2024 6:17pm

Copy link
Member

@mwest1066 mwest1066 left a comment

Choose a reason for hiding this comment

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

The thing I liked about the old order was that it made at least some attempt to explain "what is PT" at the top. Maybe we need a section somewhere towards the top titled "What is PrairieTest?"

I'm approving this PR as is.

Copy link
Contributor

@nwalters512 nwalters512 left a comment

Choose a reason for hiding this comment

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

I agree with Matt; jumping straight into value propositions without even 1 or 2 sentences about what PrairieTest is feels wrong.

I feel like the image under "Powerful exam management system" is out of place and the ideas behind it would be better communicated via plain text (either in the form of, or in addition to, some hypothetical "what is PT" text at the top of the page).

Like Matt, I'll rubber stamp this so you can merge it if you think this PR is still an improvement in its current form.

@mfsilva22
Copy link
Contributor Author

I will try to add some text. I believe the image/diagram was a request by Matt, instead of using a block of text to convey that idea. If the next update is not an improvement, we will close this PR and keep the current version, until we have time to get something better.

@mfsilva22
Copy link
Contributor Author

actually, changed my mind, it is not worth it trying to get this marginal improvement at this moment. I will close this PR now and keep the current version.

@mfsilva22 mfsilva22 closed this Mar 18, 2024
@nwalters512 nwalters512 deleted the reorganize-pt-page branch March 18, 2024 23:06
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.

3 participants