-
Notifications
You must be signed in to change notification settings - Fork 16
Grids, Tabs, and Macros #1104
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
base: main
Are you sure you want to change the base?
Grids, Tabs, and Macros #1104
Conversation
| === "External Collaborator (XIAS)" | ||
|
|
||
| The form should be prefilled as shown below. Only use the `name` portion of your XIAS email address `name@domain.tld` in the BlazerID field. Use your full XIAS email address in the Email field. | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this field is already prefilled, we could add "See incorrect or missing information? Please [contact support]." This would keep it consistent with the wording we used in the UAB Campus & UAB Medicine section.
| cards.account.responsibilities | ||
| ) | ||
| }} | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be possible to move this content into a new section? E.g, under a heading like “RCS Services for Account Holders”? Because this current page is a bit overloaded.
| renderer.render_cards( | ||
| cards.platforms.cheaha.ood.overview, | ||
| cards.platforms.cheaha.slurm.overview, | ||
| cards.data.individual_storage, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be possible to mention that XAIS will not get Globus service that has subscription by UAB?
| }} | ||
|
|
||
| === "Research Faculty Supervisor | Core Director" | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be possible to note that they may also receive all of the services provided to other users(students, staff, or XAIS guests)?
| cards.platforms.cloud_rc.web_server, | ||
| cards.account.code_rc_create, | ||
| ) | ||
| }} | indent(4) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the text “indent(4)” a command? Or was this text just typed by mistake?
| The following table has more information on each status, with links for more details. To check your status see [How Do I Check My RCS Account Status?](#how-do-i-check-my-rcs-account-status) | ||
|
|
||
| {{ read_csv("_res/status_summary.csv", keep_default_na=False) }} | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to cross-link the certification page directly in the grid and remove this table( It seems a bit redundant)? The grid already presents the necessary information in a clearer way.
|
|
||
| === "Pre-Certification" | ||
|
|
||
| When an account is in Pre-Certification status, all RCS services are accessible and action will be required soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is that to mean "..RCS services are inaccessible.... "?
|
|
||
| When an account is in Pre-Certification status, all RCS services are accessible and action will be required soon. | ||
|
|
||
| ?? days after entering Pre-Certification status, accounts will enter Certification status and action will be required. To prevent Certification status, you may certify at any time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do the double question marks represent here? Is it a typo?
|
|
||
| ?? days after entering Pre-Certification status, accounts will enter Certification status and action will be required. To prevent Certification status, you may certify at any time. | ||
|
|
||
| While in Pre-Certification status, the [Account Status page](#how-do-i-check-my-rcs-account-status) will show "Account in Pre-Certification", as shown below. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The “Account in Pre-Certification” doesn’t appear in the image. This might be confusing for users. Should we remove this text or keep it?
| - account/index.md | ||
| - Research Computing System (RCS): | ||
| - account/rcs/index.md | ||
| - Create: account/rcs/create.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed that clicking “Create” hides the menu, but the “Status” menu works correctly. Is this the intended behavior?
| - 1. Sponsor — Manage Sites: account/xias/1_sponsor_manage_sites.md | ||
| - 2. Sponsor — Manage Users: account/xias/2_sponsor_manage_users.md | ||
| - 3. Guests — Create XIAS Account: account/xias/3_guest_create_xias_account.md | ||
| - 4. Guests — Create RCS Account: account/rcs/create.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
“Create” appears in the menu under "External Collaborators" and "Research Computing System (RCS)". Would it be possible to cross- link it and keep under either of the menus?
Pull Request
Intent
A demonstration of what grids and grid cards, content tabs, and macros are capable of. The goal of this set of changes is to simplify our documentation's presentation and reduce the cognitive load of readers, while important making content easier to discover.
Proposed Changes
index.mdres/announcements.ymlhelp/support.mdaccount_managementsection of the documentationaccountfor shorter URLsmkdocs-macrospluginmkdocs-table-reader-pluginfor compatibility withmkdocs-macrosbuild_env.ymlfor compatibility fixes/docs/_macro/(using jinja2)/macros/(grid card engine using YAML data)_templatedirectories with jinja2 templates supporting the documentation in the same directoryresto_resfor better visibilityimagesto_imgfor better visibility{% raw %}blocks where necessary to avoid unexpected jinja2 template interpretationres/grid_cards.ymlChanges to Section Headers
There are many, we will need to carefully review the changes.
Changes to Page URLs
There are many, we will need to carefully review the changes.
Related or Fixed Issues
Related to #858, #1094, #1095,
Accessibility Checklist
I have ensured all of the following for new or changed content:
Style Checklist
I have done all of the following:
mkdocs.ymlfor any moved headers or pages.