Skip to content

Polish/UI#1

Merged
ocl2go merged 10 commits intomainfrom
polish/ui
Oct 28, 2025
Merged

Polish/UI#1
ocl2go merged 10 commits intomainfrom
polish/ui

Conversation

@ocl2go
Copy link
Copy Markdown
Contributor

@ocl2go ocl2go commented Oct 28, 2025

概要

いい感じに修正

確認

元のコード: `var e="onekg-jn@sfc.keio.ac.jp",t=document.getElementById("email");t&&(t.textContent=e,t.href="mailto:"+e);`

(採用しなかったが)さらなる短縮案: `with(email)href="mailto:"+(text="onekg-jn@sfc.keio.ac.jp")`
- Remove OGP meta tags (4 lines): contradicts design philosophy
- Adjust table indent: single space → 3x nbsp (1.274em)
- Remove blank lines (3 lines)
- Size: 31,421 → 31,161 bytes (-0.8%)
- Change table first cell from     to empty (4 places)
- Empty td creates 0.375em (6px) indent without CSS
- Tested 20+ HTML alternatives: all failed (0em or 2.5em)
- Table spec stable for 25+ years, works for next 10+ years
- Meta: OS・ネットワークを「基盤とした」と表現
- Header: 「など広範な」で柔軟性を保持
- About: 「基盤に」でアイデンティティ明記、エッジ・分散・DB・Webも含む
- 研究室の歴史(OS・ネットワーク)と最近の研究(エッジ等)のバランスを実現
@ocl2go ocl2go requested a review from sakusaku3939 October 28, 2025 09:50
Copy link
Copy Markdown
Member

@sakusaku3939 sakusaku3939 left a comment

Choose a reason for hiding this comment

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

ありがとう!!めちゃめちゃ良い
細かいとこだけど気になったところだけ

Comment thread index.html Outdated
Comment thread index.html Outdated

<footer>
<p><small>
Contact: yanma / Email: <a id="email"></a><br>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

imo
迷ったけど、KG Leaderって文言はあった方が参加希望の人が問い合わせしやすそうで、そうなるとContactは元々あった1セクションとしてh2タグに分けた方が見やすいかも、?
もし変えた理由とかあれば聞きたい!

Copy link
Copy Markdown
Contributor Author

@ocl2go ocl2go Oct 28, 2025

Choose a reason for hiding this comment

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

自分の感性で申し訳ないのだけど、footer付近に素のh2タグだとダサく感じたんだよね。

KGLであることを明記するなら、header付近にKGL: yanmaとか書いておくのもありかも。

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

あーーなるほど確かに
ただ、Researchの中に含まれちゃってる感じがして、それなら© 2025 Nakazawa and Okoshi Laboratoryを消してfooterごとContactセクションに入れるのはアリだと今思った

Copy link
Copy Markdown
Member

@sakusaku3939 sakusaku3939 Oct 28, 2025

Choose a reason for hiding this comment

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

(ちなみにWeb業界的には© 2025 〜自体に法的効力は無く、むしろずっと更新が止まってるサイトを見分けるだけのものになってる話もある)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

なるほど!
たしかにコピーライトはロマン説も全然あるね
とりあえず一旦何個か試してみて良さげなのコミットしてみます

@sakusaku3939
Copy link
Copy Markdown
Member

これPR投げた時にhtmlpreviewをGitHub Actionsで自動でコメント残すやつ入れたいな

@sakusaku3939
Copy link
Copy Markdown
Member

これPR投げた時にhtmlpreviewをGitHub Actionsで自動でコメント残すやつ入れたいな

入れてみた 🚀
https://github.com/nkzwlab/sysnet/blob/main/.github/workflows/preview.yml

@ocl2go
Copy link
Copy Markdown
Contributor Author

ocl2go commented Oct 28, 2025

@sakusaku3939
ないす!なんかfailしてるから見てほしい。
あと、デザインで、contactとresearchのdetailsの中を変えたから感想ほしい

@sakusaku3939
Copy link
Copy Markdown
Member

sakusaku3939 commented Oct 28, 2025

すごいリッチになってる!良き
Abstractの枠もfooterの線もめっちゃ良い感じ~

PreviewのCIジョブの件は修正できたはずだから気にしないで(一応適当なコミット入れてくれれば次は通るはず

Copy link
Copy Markdown
Member

@sakusaku3939 sakusaku3939 left a comment

Choose a reason for hiding this comment

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

zp7m284va81g3uz7tlduujp3

@ocl2go
Copy link
Copy Markdown
Contributor Author

ocl2go commented Oct 28, 2025

ありがとう!mergeするね

@sakusaku3939
Copy link
Copy Markdown
Member

一回適当なコミット試してみていい?

@ocl2go ocl2go merged commit e1ec341 into main Oct 28, 2025
1 check failed
@ocl2go
Copy link
Copy Markdown
Contributor Author

ocl2go commented Oct 28, 2025

@sakusaku3939
Copy link
Copy Markdown
Member

いや俺が悪いからOK(次のPR作って試すw)

@ocl2go
Copy link
Copy Markdown
Contributor Author

ocl2go commented Oct 28, 2025

ごめん!全然好きにcommitしていいと思う。

@sakusaku3939 sakusaku3939 deleted the polish/ui branch October 28, 2025 15:30
@sakusaku3939
Copy link
Copy Markdown
Member

@ocl2go
うまく行ってた #2

ちなみにこれは全然話変わるけど、このリポジトリのライセンスはHot poteto licenseだから今この瞬間に全ての権限がyanmaに渡ったということで ↓
https://github.com/nkzwlab/sysnet/blob/main/hot-potato-license

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