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

Project gallery -> Projects #18135

Merged
merged 8 commits into from Oct 6, 2017
Merged

Project gallery -> Projects #18135

merged 8 commits into from Oct 6, 2017

Conversation

Erin007
Copy link
Contributor

@Erin007 Erin007 commented Oct 4, 2017

This PR changes "Project Gallery" to "Projects" in the navbar, hamburger, projects page title and link to the projects page from /home. It also moves "Projects" in the navbar list and adds a subheading to the ProjectHeader.

Signed out projects before:
signed out projects before

Signed out projects after:
signed out projects after

Signed in projects before:
signed in projects before

Signed in projects after:
signed in projects after

hamburger dropdown before:
hamburger before

hamburger dropdown after:
hamburger after

/home link before:
view project gallery link before

/home link after:
view projects link after

@Erin007
Copy link
Contributor Author

Erin007 commented Oct 4, 2017

cc: @tanyaparker

@Erin007
Copy link
Contributor Author

Erin007 commented Oct 4, 2017

I also changed the string in the i18n gsheet for pegasus from "Project Gallery" to "Projects".

"projectsViewOldGallery": "View old gallery with other project types",
"projects": "Projects",
"projectsSubHeading": "Students have created over 20 million projects",
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, didn't update TODO list with the copy. The string should be "Over 20 million projects created"

Is it possible to do formatting and somehow make the 20 million more bold and more big?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We can either have a localized string OR we can style the 20 million differently than the rest of the string. My understanding is that for translations to be successful we can't translate substrings and then put them back together because maybe the ordering matters in different languages, etc. What's your preference?

Copy link
Contributor

Choose a reason for hiding this comment

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

Merp. Ok, let's go with localized > formatting.

Copy link
Member

@davidsbailey davidsbailey left a comment

Choose a reason for hiding this comment

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

I don't see header_project_gallery (the name of the key for the string "Project Gallery" in the pegasus gsheet) used anywhere in the codebase. Perhaps worth simply removing from the gsheet? OTOH I can't find many of the strings from that gsheet in the codebase, so it's possible I'm missing something.

@Erin007 Erin007 force-pushed the project-gallery-link-updates branch from 7e64ebe to 50b9868 Compare October 5, 2017 23:27
@Erin007 Erin007 merged commit e793fb6 into staging Oct 6, 2017
@Erin007 Erin007 deleted the project-gallery-link-updates branch October 6, 2017 03:51
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.

None yet

3 participants