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

Update branding and copyrights #102

Merged
merged 2 commits into from
Sep 12, 2024
Merged

Update branding and copyrights #102

merged 2 commits into from
Sep 12, 2024

Conversation

mint-thompson
Copy link
Collaborator

Remove MITRE logo from header.

Remove MITRE section from about page. Add HL7 section to about page. Change copyright to HL7. Change maintainers to HL7 community.

Remove MITRE logo from header.
Remove MITRE section from about page. Add HL7 section to about page.
Change copyright to HL7. Change maintainers to HL7 community.
@@ -27,7 +27,7 @@ To develop this site locally:
$ git submodule update --init --recursive
```
2. [Install Hugo](https://gohugo.io/getting-started/installing/)
3. Add and edit content, refferring to the [Hugo documentation](https://gohugo.io/documentation/)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Lol wooo! 🎉

Copy link
Member

@cmoesel cmoesel left a comment

Choose a reason for hiding this comment

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

This is looking good. I found a couple of things that I think we should fix before merging:

  1. The HL7 logo on the About page is too small (see individual comment below).
  2. The footer still contains the PRS statement. That should be removed (see screenshot below).

CleanShot 2024-09-12 at 08 38 53@2x

Copy link
Member

Choose a reason for hiding this comment

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

This HL7 logo looks funny next to the larger logos on the About page:
CleanShot 2024-09-12 at 08 32 51@2x

I found a bigger version that I think would look better: https://fire.ly/wp-content/uploads/2020/11/HL7_int_01.png.webp

Bigger HL7 Logo

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thank you for finding this bigger logo! It definitely looks much better alongside the other two. I've updated the PR to use it.

Copy link
Member

@cmoesel cmoesel left a comment

Choose a reason for hiding this comment

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

Looks good. Thank you, Mint!

@mint-thompson mint-thompson merged commit 97f707a into master Sep 12, 2024
@mint-thompson mint-thompson deleted the update-branding branch September 12, 2024 17:54
@mint-thompson mint-thompson restored the update-branding branch September 12, 2024 17:56
@mint-thompson mint-thompson deleted the update-branding branch September 12, 2024 17:56
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