Skip to content

ダウンロードしたhtmlのボタンを非表示にする#40

Open
hatosh wants to merge 15 commits into
masterfrom
button
Open

ダウンロードしたhtmlのボタンを非表示にする#40
hatosh wants to merge 15 commits into
masterfrom
button

Conversation

@hatosh

@hatosh hatosh commented Apr 22, 2020

Copy link
Copy Markdown
Contributor

No description provided.

@ahuglajbclajep ahuglajbclajep left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

display = "none" でやったほうがいいんですが、これをやるとメモ欄など一部スタイルが崩れたりするので、全てのボタンに対して display = "none" するのではなく一部は visibility = "hidden" でやるとか、そもそも .graph-menu 自体がいらないとかもあるので、全体的にちょっと調整は必要そう。

Comment thread src/lib/util.ts Outdated
Comment thread src/lib/util.ts Outdated
@ahuglajbclajep ahuglajbclajep added the tech: CSS CSS 関連 label Apr 24, 2020

@ahuglajbclajep ahuglajbclajep left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

今後UIが複雑になってくるとCSSなりで宣言的にやったほうがいい気もするけど、結局消す部分にラベルをつけるとかだけでは細かい部分を調整できなさそうなので、とりあえずは今のやり方で進めて行くつもり。

Comment thread src/lib/util.ts
Comment thread src/lib/util.ts Outdated
Comment thread src/lib/util.ts Outdated
@ahuglajbclajep

ahuglajbclajep commented May 11, 2020

Copy link
Copy Markdown
Contributor

まあ style-loader 使っててCSSのminifyやってないので、やはり特定のCSSだけ raw-loader で文字列として読んで差し込むとかが一番なのかなぁという気はしてきた。
とりあえずこのPRではDOMを直接いじってそれでこんな感じにするというのは固めて、実際に raw-loader 使うかとかは別のPRで考えることにします。

@ahuglajbclajep ahuglajbclajep added the category: refactor リファクタリングの話 label Jul 24, 2020
@ahuglajbclajep

ahuglajbclajep commented Aug 28, 2020

Copy link
Copy Markdown
Contributor

#73 の実装。
ちょっと期間が空いてしまったので改めて見ておきます。

@ahuglajbclajep

ahuglajbclajep commented Sep 13, 2020

Copy link
Copy Markdown
Contributor

ビルドが通らなかったのでとりあえず pull -r origin master した。
でもまあ #45 がマージされてから手をつけた方が良さそうですね……

@ahuglajbclajep

ahuglajbclajep commented Nov 26, 2020

Copy link
Copy Markdown
Contributor

あとは doc.getElementsByTagName("script")[0];doc.getElementsByTagName("canvas")[0]etElementsByClassName() で書き直せばとりあえずは大丈夫そう。
本当は getElementById() を使うべきだがこれは Element には生えていないので使えないのだった。
あとは graph-canvas の下の角を丸くするとか。

見た目の調整は結局こういう細かい操作があるので、宣言的に書くのはなかなか難しそう。

@ahuglajbclajep ahuglajbclajep added category: enhancement 新機能の話 and removed category: refactor リファクタリングの話 category: enhancement 新機能の話 labels Mar 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tech: CSS CSS 関連

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants