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

Issue #485 - website refactor #486

Merged
merged 50 commits into from Jan 23, 2022
Merged

Issue #485 - website refactor #486

merged 50 commits into from Jan 23, 2022

Conversation

organom
Copy link
Member

@organom organom commented Sep 27, 2021

  • Polished website design
  • Updated social and documentation links
  • Quick start area refactor

Fixes #485

@organom organom requested a review from ilyash September 27, 2021 12:34
@organom
Copy link
Member Author

organom commented Sep 28, 2021

@ilyash This sentence is not clear: If you need additional function or syntax in NGS, it's highly probable that it will be included in the language as long as it is for DevOps tasks. please provide alternative

@organom
Copy link
Member Author

organom commented Sep 28, 2021

@ilyash search icon for the Language reference - please choose one from https://fontawesome.com/v5.15/icons

@organom
Copy link
Member Author

organom commented Sep 28, 2021

image

Latest design

… new marketing-material logo location and removed .svg
Copy link
Member

@ilyash ilyash left a comment

Choose a reason for hiding this comment

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

  • I like the top section (above logos). Very clean.
  • Logos are good but I would consider converting the two left to textual links.
  • The left two logos should open in same tab, not new - it's the same site and opening in new tab feels weird
  • The site breaks when width is small, especially logos section
  • The logos in "Quick Start" section don't feel right. Maybe smaller and/or without frames?

How do you know that site upload script would handle the directories (img)?

@organom
Copy link
Member Author

organom commented Oct 4, 2021

  • I like the top section (above logos). Very clean.
  • Logos are good but I would consider converting the two left to textual links. >>>>> would break design :'(
  • The left two logos should open in same tab, not new - it's the same site and opening in new tab feels weird
  • The site breaks when width is small, especially logos section
  • The logos in "Quick Start" section don't feel right. Maybe smaller and/or without frames?

How do you know that site upload script would handle the directories (img)? >>>>> With the new implementation we don't need the folder anymore, since we don't use the logos. Moved img folder into marketing_material repository to use in case we ever need that logos

@organom
Copy link
Member Author

organom commented Oct 4, 2021

Screenshot 2021-10-04 at 23 24 20
Screenshot 2021-10-04 at 23 25 20

@organom organom requested a review from ilyash October 4, 2021 22:30
@organom
Copy link
Member Author

organom commented Oct 6, 2021

Added meeting information. @ilyash does this fit your schedule?

Fixes #487
The issue i see with zoom is that or we use your private id, or then every week the meeting will be in a different address. I propose we try the discord meeting channel i created for that and use youtube live for recording/broadcasting. Lets try it this weekend, even if in a very fast matter :)

Screenshot 2021-10-06 at 15 17 40

Copy link
Contributor

@ilyash-b ilyash-b left a comment

Choose a reason for hiding this comment

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

  1. Please bring back the logo to where it was. It should be served from the same domain. We don’t start using GitHub as CDN just to serve couple of files.
  2. I have finally given some thought to the subdomains. Any redirects should not be used. It’s just worse user experience due to additional redirect. Use direct links. Additionally, some redirects are broken but it’s not important already.
  3. Remove “_blank” targets. I think the behavior is surprising.

</p>
</div>
<div class="col-md-3">
</div>
<div class="row pt-3 pb-3">
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, let's try this. Maybe on a different branch?

site/index.html Outdated Show resolved Hide resolved
@organom
Copy link
Member Author

organom commented Dec 22, 2021

  1. Please bring back the logo to where it was. It should be served from the same domain. We don’t start using GitHub as CDN just to serve couple of files.

  2. I have finally given some thought to the subdomains. Any redirects should not be used. It’s just worse user experience due to additional redirect. Use direct links. Additionally, some redirects are broken but it’s not important already.

  3. Remove “_blank” targets. I think the behavior is surprising.

@ilyash Made the above requested changes. The menu as a "bar" is available in feature/website-refactor-2 branch, please let me know the one you prefer

@organom organom changed the title Feature - website refactor #485 - website refactor Jan 9, 2022
@organom organom changed the title #485 - website refactor Issue #485 - website refactor Jan 9, 2022
@ilyash-b
Copy link
Contributor

@organom , I liked and therefore merged feature/website-refactor-2

@ilyash-b
Copy link
Contributor

@organom , please make sure that the page passes validation. Right now it doesn't.

@ilyash-b
Copy link
Contributor

@organom , please make sure to use integrity and pinned versions in all script elements

@organom
Copy link
Member Author

organom commented Jan 18, 2022

@organom , please make sure that the page passes validation. Right now it doesn't.

@ilyash-b What validator shall I use?

@organom
Copy link
Member Author

organom commented Jan 19, 2022

@organom , please make sure that the page passes validation. Right now it doesn't.

@ilyash-b What validator shall I use?

Used W3C validator. Please re-review and confirm

Comment on lines +31 to +49
REDIRECTS= [{id: "discord", host: "discord.gg", path: "6VqgcpM"},
{id: "videos", host: "m.youtube.com", path: "channel/UC6CL2J6syrV1mfFpxQd89gQ"},
{id: "source", host: "github.com", path: "ngs-lang/ngs"},
{id: "wiki", host: "github.com", path: "ngs-lang/ngs/wiki"},
{id: "ipython", host: "github.com", path: "ngs-lang/ngs-ipython-extension"},
{id: "docker", host: "hub.docker.com", path: "r/ngslang/ngs"},
{id: "brew", host: "formulae.brew.sh", path: "formula/ngs"}
{id: "snap", host: "snapcraft.io", path: "ngs"},
{id: "action", host: "github.com", path: "marketplace/actions/setup-ngs"},
{id: "manual", host: "ngs-lang.org", path: "doc/latest/index.html"},
{id: "ref", host: "ngs-lang.org", path: "doc/latest/generated/index.html"}]

REDIRECTS.each({
bucketName = "${A.id}.${DOMAIN}"
tmpbucket = AWS::Bucket(Name=bucketName).converge()
if not($(ok:[0, 255] aws s3api get-bucket-website --bucket ${bucketName})) {
tmpbucket.resources[0].run("Setting website", %(aws s3 website "s3://${bucketName}/" --index-document index.html --error-document error.html ), false)
}
})
Copy link
Contributor

Choose a reason for hiding this comment

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

Still relevant?

Copy link
Member Author

Choose a reason for hiding this comment

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

@ilyash-b maybe not all of them, but since they still exist (even if not used on the website) I would leave it. I'm at least using all the time some of them to go directly to the right place :)

@organom organom requested a review from ilyash-b January 23, 2022 15:40
@ilyash-b ilyash-b merged commit 939bca2 into dev Jan 23, 2022
@organom organom deleted the feature/website-refactor branch January 23, 2022 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants