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 pages and Get Involved pages minor adjustments #6743

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

goplayoutside3
Copy link
Contributor

@goplayoutside3 goplayoutside3 commented Mar 6, 2025

Package

lib-content

Linked Issue and/or Talk Post

Closes: #6688

Describe your changes

  • See the list of minor copy updates and design adjustments in the above Issue
  • One more thing I caught and fixed in this PR
    • The "All" link on the Publications page was not scrolling the page back up to the top. I decided to remove it (because the Teams page does not have an All link) and because its label did not clearly indicate which part of the webpage it was supposed to take you to. There is no section labeled "All" with a header.

How to Review

  • Run app-root locally and visit the /about and /get-involved pages
  • For the one adjustment to /groups, go to any individual stats page or groups page and filter the bar chart to a time period in which there are no classifications to see the "no data" message. Example:

Checklist

PR Creator - Please cater the checklist to fit the review needed for your code changes.
PR Reviewer - Use the checklist during your review. Each point should be checkmarked or discussed before PR approval.

General

  • Tests are passing locally and on Github
  • Documentation is up to date and changelog has been updated if appropriate
  • You can yarn panic && yarn bootstrap or docker-compose up --build and FEM works as expected
  • FEM works in all major desktop browsers: Firefox, Chrome, Edge, Safari (Use Browserstack account as needed)

(See the checklist in the linked Issue)

@goplayoutside3 goplayoutside3 changed the title About pages and Get Involved pages minor adjustements About pages and Get Involved pages minor adjustments Mar 6, 2025
@goplayoutside3 goplayoutside3 requested a review from mcbouslog March 6, 2025 19:40
@coveralls
Copy link

Coverage Status

coverage: 75.304% (-0.04%) from 75.345%
when pulling 51d453d on about-page-adjustments
into ee36b03 on master.

@mcbouslog mcbouslog self-assigned this Mar 11, 2025
Copy link
Contributor

@mcbouslog mcbouslog left a comment

Choose a reason for hiding this comment

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

LGTM! Confirmed changes made per linked issue. 🎨

@@ -94,7 +94,7 @@
"paragraphs": {
"one": "The Zooniverse is the world's largest and most popular platform for people-powered research. This research is made possible by volunteers—millions of people around the world who come together to assist professional researchers. Our goal is to enable research that would not be possible, or practical, otherwise. Zooniverse research results in new discoveries, datasets useful to the wider research community, and many <0>publications</0>.",
"two": "You don't need any specialized background, training, or expertise to participate in Zooniverse projects. We make it easy for anyone to contribute to real academic research, on their own computer, at their convenience. You'll be able to study authentic data gathered by researchers, like images of faraway galaxies, historical records and diaries, or videos of animals in their natural habitats. By participating in projects, you'll help contribute to our understanding of the world, the universe, and more. With our wide-ranging and ever-expanding suite of projects, which cover many disciplines and topics across the sciences and humanities, there's a place for anyone and everyone to explore, learn, and have fun in the Zooniverse. To volunteer, visit the <0>Projects page</0>, choose one you like the look of, and get started.",
"three": "The major challenge of 21st century research is dealing with the flood of information we can now collect about the world around us. Computers can help, but in many fields the human ability for pattern recognition—and our ability to be surprised—makes us superior. With the help of Zooniverse volunteers, researchers can analyze their information more quickly and accurately than would otherwise be possible, getting to exciting results more quickly. Your effort saves time and resources, advances the ability of computers to do the same tasks, and leads to faster progress and understanding of the world. Our projects combine contributions from multiple individual volunteers, relying on a version of the <0>'wisdom of the crowds'</0> to produce reliable and accurate results. Having many people look at the data often leads to better estimates of how likely we are to make an error. The products of Zooniverse projects are often exactly what's needed to make progress in many fields of research.",
"three": "The major challenge of 21st century research is dealing with the flood of information we can now collect about the world around us. Computers can help, but in many fields the human ability for pattern recognition—and our ability to be surprised—makes us superior. With the help of Zooniverse volunteers, researchers can analyze their information more quickly and accurately than would otherwise be possible, getting to exciting results more quickly. Your effort saves time and resources, advances the ability of computers to do the same tasks, and leads to faster progress and understanding of the world. Our projects combine contributions from multiple individual volunteers, relying on a version of the <0>'wisdom of the crowd'</0> to produce reliable and accurate results. Having many people look at the data often leads to better estimates of how likely we are to make an error. The products of Zooniverse projects are often exactly what's needed to make progress in many fields of research.",
Copy link
Contributor

Choose a reason for hiding this comment

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

This is overly pedantic, but the heading notes "The wisdom of the crowd" , including the within the quotes, compared to this paragraph where the is outside the quotes. Maybe the in this paragraph should be included within the quotes to match the heading.

@github-actions github-actions bot added the approved This PR is approved for merging label Mar 12, 2025
@mcbouslog mcbouslog removed their assignment Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved This PR is approved for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Groups, About, and Get-Involved pages: quick copy fixes
3 participants