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

Put meeting schedule into table #831

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

Conversation

bact
Copy link
Contributor

@bact bact commented Dec 23, 2024

  • Arguably more visually organized
  • Add link to https://www.time.gov/ for US Eastern Time check
  • Also add links to minutes folders

The new layout will rendered like this:

Team Meetings

Profile Meetings

Focus Group Meetings

bact added 2 commits December 23, 2024 06:26
- Arguably more visually organized
- Also add links to minutes folders

Signed-off-by: Arthit Suriyawongkul <[email protected]>
Signed-off-by: Arthit Suriyawongkul <[email protected]>
@goneall
Copy link
Member

goneall commented Dec 23, 2024

One issue with the new organization is that you can no longer link to the specific section for a meeting (e.g. Tech Team Meetings).

Agree it is more readable, but I'm wondering if it's worth loosing the links. BTW - I only use those links in emails myself, I don't know if there are any websites which use the links.

@bact
Copy link
Contributor Author

bact commented Dec 24, 2024

Very good point. I will figure if we can keep the links untouched.

Signed-off-by: Arthit Suriyawongkul <[email protected]>
Signed-off-by: Arthit Suriyawongkul <[email protected]>
@bact
Copy link
Contributor Author

bact commented Dec 24, 2024

I have proposed another layout at #833.

It sticks with traditional style of headings + keep old links,
while info are in individual tables that share same format for table heading.

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.

2 participants