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

about page #1533

Merged
merged 19 commits into from
Feb 2, 2024
Merged

about page #1533

merged 19 commits into from
Feb 2, 2024

Conversation

varCepheid
Copy link
Collaborator

closes #1531
This PR creates an about page, linked from the sidebar menu, that has information about what Peregrine is, how teams can start using it or get help using it, and some useful links and contact information.

@varCepheid varCepheid self-assigned this Aug 2, 2023
@varCepheid varCepheid added enhancement Adds a new feature or meets a request. wip priority This needs to be addressed before the current changes can be merged to main. labels Aug 2, 2023
@varCepheid varCepheid marked this pull request as draft August 2, 2023 17:16
@varCepheid
Copy link
Collaborator Author

I put in a version number, but I can easily take that out if we don't like it. Also, whoever reviews this, please proofread the text.

@varCepheid varCepheid marked this pull request as ready for review August 2, 2023 18:52
@varCepheid varCepheid removed the wip label Aug 2, 2023
Copy link
Member

@calebeby calebeby left a comment

Choose a reason for hiding this comment

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

@varCepheid Thanks for making this PR! I've wanted to have some sort of homepage/intro page for a long time!

src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
@calebeby calebeby added the needs work Changes have been suggested and need to be implemented. label Aug 9, 2023
replaced grid with constrained widths
added semantic HTML tags
linked email address
added header tag
@varCepheid varCepheid added question Further information is requested. and removed needs work Changes have been suggested and need to be implemented. labels Nov 8, 2023
@varCepheid varCepheid removed the question Further information is requested. label Nov 8, 2023
Copy link
Member

@calebeby calebeby left a comment

Choose a reason for hiding this comment

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

@varCepheid Thanks for making this about page, it is definitely something that I've wanted to exist for a long time but was never able to prioritize. This PR is getting pretty close to being ready!

src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
@calebeby calebeby added the needs work Changes have been suggested and need to be implemented. label Nov 12, 2023
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
@varCepheid varCepheid removed the needs work Changes have been suggested and need to be implemented. label Nov 15, 2023
@calebeby calebeby self-requested a review November 19, 2023 06:39
Copy link
Member

@calebeby calebeby left a comment

Choose a reason for hiding this comment

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

Left some minor comments, looking pretty good

src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
src/routes/about.tsx Outdated Show resolved Hide resolved
calebeby
calebeby previously approved these changes Jan 6, 2024
Copy link
Member

@calebeby calebeby left a comment

Choose a reason for hiding this comment

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

Couple more tiny suggestions. I am happy with this, feel free to merge once you've addressed these!

<p>
<strong>{"It's really easy to get started with Peregrine."}</strong>
{' Have a team captain or coach send an email to '}
<a href="mailto:[email protected]"> [email protected] </a>
Copy link
Member

Choose a reason for hiding this comment

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

The space is linked here, which I think was not intended:

image

Suggested change
<a href="mailto:[email protected]"> [email protected] </a>
<a href="mailto:[email protected]">[email protected]</a>

(not sure why the space after is linked but not the space before. Usually for HTML elements you do not want to have spaces right inside of the element declaration, so I'd suggest making this kind of change throughout this file.

}
</strong>
{'at '}
<a href="mailto:[email protected]"> [email protected]. </a>
Copy link
Member

Choose a reason for hiding this comment

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

Nit: The ending period is linked, which is not part of the email:

image

Suggested change
<a href="mailto:[email protected]"> [email protected]. </a>
<a href="mailto:[email protected]">[email protected]</a>{'.'}

(not sure how prettier will want to format this)

@calebeby calebeby self-requested a review February 2, 2024 23:49
@varCepheid varCepheid added this pull request to the merge queue Feb 2, 2024
Merged via the queue into dev with commit c5ee369 Feb 2, 2024
12 checks passed
@varCepheid varCepheid deleted the about-page branch February 2, 2024 23:51
github-merge-queue bot pushed a commit that referenced this pull request Mar 8, 2024
* created about page and added link from sidebar menu

* created the route

* changed export to default

* added more substance to about page

* added text and styling

* changed to grid layout and some editing

* aesthetic changes

* revisions to the about blurb

* updated about page

replaced grid with constrained widths
added semantic HTML tags
linked email address
added header tag

* fixed text width, header margins

* padding and margin changes

* editing and minor changes

* replaced text with embedded expressions

* adjusted width parameters to make text narrower

* formatting changes

* convention & hyperlink nits

---------

Co-authored-by: Caleb Eby <[email protected]>
Co-authored-by: Caleb Eby <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Adds a new feature or meets a request. priority This needs to be addressed before the current changes can be merged to main.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

about page
2 participants