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

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

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

hatosh
Copy link
Contributor

@hatosh hatosh commented Apr 22, 2020

No description provided.

Copy link
Collaborator

@ahuglajbclajep ahuglajbclajep left a comment

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 自体がいらないとかもあるので、全体的にちょっと調整は必要そう。

src/lib/util.ts Outdated Show resolved Hide resolved
src/lib/util.ts Outdated Show resolved Hide resolved
@ahuglajbclajep ahuglajbclajep added the tech: CSS CSS 関連 label Apr 24, 2020
Copy link
Collaborator

@ahuglajbclajep ahuglajbclajep left a comment

Choose a reason for hiding this comment

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

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

src/lib/util.ts Show resolved Hide resolved
src/lib/util.ts Outdated Show resolved Hide resolved
src/lib/util.ts Outdated Show resolved Hide resolved
@ahuglajbclajep
Copy link
Collaborator

ahuglajbclajep commented May 11, 2020

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

@ahuglajbclajep
Copy link
Collaborator

ahuglajbclajep commented Aug 28, 2020

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

@ahuglajbclajep
Copy link
Collaborator

ahuglajbclajep commented Sep 13, 2020

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

@ahuglajbclajep
Copy link
Collaborator

ahuglajbclajep commented Nov 26, 2020

あとは 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