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 web bubble for credentials #14306

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

usamabinnadeem-10
Copy link
Contributor

Done

  • Update the web bubble for credentials

QA

  • Check out this feature branch
  • Run the site using the command ./run serve or dotrun
  • View the site locally in your web browser at: http://0.0.0.0:8001/
    • Be sure to test on mobile, tablet and desktop screen sizes
  • Go to this spreadsheet for suggested UI changes

Issue / Card

Fixes #14502

Help

QA steps - Commit guidelines

@webteam-app
Copy link

@linux-in-flux
Copy link
Collaborator

credentials page:

  • My understanding of Ege's note was to not use an exclamation point here, and she preferred "Get the exam" to "Take the exam". I would also suggest this, as they are purchasing it but not necessarily taking it immediately.
  • Still has "Get involved" section although button was merged with above section.

Syllabus page

  • should be renamed to "Exam Content" (this was in a comment, but I don't think Ege updated the document.)
  • Top copy should be shortened per E2
  • Should have "Get Certification" button per E2
  • Subitems should be bullet points instead of numbers (see E3)
  • Check with Ege, as the

Self-Study (aka Study Guide):

  • Tabs have been made horizontal but don't match style from Syllabus page as requested

FAQ:

  • Sidebar links are shortened but section titles are not. Clarify intention with Ege.
  • Width is still inconsistent at "Before..." Transition (see E4)

Your-exams E3 is unverified because I have a million exams and don't know what it looks like w/o.

Shop

  • Coming Soon needs to be corrected to Coming soon (no capitalization), check with Ege on exclamation point. She didn't say no here, but she said to be strict with exclamation point usage.

Checkout

  • D3 was a little unclear, but I believe what is being asked is to remove "* indicates a mandatory field" from the top and put it underneath each mandatory field. Also, Region should be marked as a mandatory field. Note that this would also impact Ubuntu Pro, so you may want to check with them, but it sounded like she already talked to them?

@usamabinnadeem-10
Copy link
Contributor Author

usamabinnadeem-10 commented Sep 20, 2024

@linux-in-flux should I remove the "Get involved" section?
For the rename to "Exam content", you are referring to the navigation tab?

@linux-in-flux
Copy link
Collaborator

For the "Get involved" section, the design review says "We can merge the section below [Get involved] to this one [SME/Tester], it doesn't need its section. The amount of space it occupies isn't justified against the amount of information it conveys." So yes, the copy changes made in the SME section and addition of Contact Us button make the Get involved section redundant, please remove.

"Exam content":

  • Across bubble, change navigation button.
  • On credentials main page change links
  • On [currently Syllabus page] change titles at top of page and <title>.
    Eliminate the word "Syllabus" from bubble due to translation issues, per Cedric.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants