Docs: Add meeting calendar and remove deprecated contact methods#76
Conversation
Signed-off-by: Ed Snible <snible@us.ibm.com>
mrsabath
left a comment
There was a problem hiding this comment.
Clean, well-scoped change — the calendar lands in the right column and the dead GitHub Discussions links are correctly removed from both spots. Links keep rel="noopener noreferrer", URLs are properly entity-encoded, and the markup is well-formed. CI green, signed-off, Resolves #73.
Two minor polish items inline (non-blocking): the embedded calendar iframe uses a fixed width="800" on an otherwise responsive page, so it'll overflow on mobile; and a title attribute would help screen readers. Both fine to fold in now or leave for a follow-up.
Areas reviewed: HTML/Docs, security (links/iframe), conventions, CI · Commits: 1, signed-off: yes · CI: passing
| <div class="community-cols"> | ||
|
|
||
| <div class="community-col"> | ||
| <h3 class="community-col-label">Community meetings</h3> |
There was a problem hiding this comment.
suggestion: this iframe has a fixed width="800". The page sets a mobile viewport (width=device-width) and the rest of the layout is responsive, so on narrow screens this will overflow horizontally. Consider dropping the fixed width in favor of style="border:0; max-width:100%" (or wrapping it in a responsive container) so it scales down on mobile.
| <div class="community-cols"> | ||
|
|
||
| <div class="community-col"> | ||
| <h3 class="community-col-label">Community meetings</h3> |
There was a problem hiding this comment.
nit: consider adding title="Kagenti community calendar" to the <iframe> — screen readers announce the title, and it's a small accessibility win.
Summary
Adds the Community Meeting calendar to the web site.
Removes deprecated contact methods (GitHub discussions)
(Optional) Testing Instructions
Bring this PR into your workspace and open index.html in a browser.
Fixes #
Resolves #73