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.

Sign up for GitHub

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

Jump to bottom

Improve paste uploading Resolve #3023 #4542

Merged
merged 12 commits into from
Jul 8, 2019
Merged

Improve paste uploading Resolve #3023 #4542

merged 12 commits into from
Jul 8, 2019

Conversation

tamaina
Copy link
Contributor

@tamaina tamaina commented Mar 19, 2019 •
edited
Loading

Summary

resolve #3023

画像をペーストした際にこれまではimage.pngという名前で保存されてimage.pngがたくさんあふれてしまっていたのを、日付時刻による名付けを行えるように。

設定 > 動作で以下のオプションを変更可能にした。

  • テンプレートを設定できるように（moment.jsで指定可能な形式+[{{number}}]で連番指定）
  • ペーストした時点で名前を聞くように
  • ペーストした画像を保存するフォルダーを指定可能に（ルート、もしくはルート直下のフォルダの中から）
    • （フォルダー選択ダイアログがあればそちらのほうが良いと思う） フォルダー選択ダイアログにした

https://youtu.be/aGF59wffkvE

@tamaina
Copy link
Contributor Author

tamaina commented Mar 26, 2019

フォルダー選択ダイアログの存在を見つけた

@tamaina tamaina changed the title resolve #3023 WIP: resolve #3023 Mar 26, 2019
@tamaina
Copy link
Contributor Author

tamaina commented Mar 26, 2019 •
edited
Loading

ぬっ、UIがめんどくさい（いったんやめておく）

@tamaina tamaina changed the title WIP: resolve #3023 resolve #3023 Mar 26, 2019
@tamaina tamaina changed the title resolve #3023 WIP: resolve #3023 Mar 26, 2019
@tamaina
Copy link
Contributor Author

tamaina commented Mar 26, 2019

UIを思いついたので再びWipにします

@tamaina tamaina changed the title WIP: resolve #3023 Improve Paste Uploading Resolve #3023 Mar 27, 2019
@tamaina tamaina changed the title Improve Paste Uploading Resolve #3023 Improve paste uploading Resolve #3023 Mar 27, 2019
@tamaina
Copy link
Contributor Author

tamaina commented Mar 27, 2019

とりあえずできた

@tamaina
Copy link
Contributor Author

tamaina commented Apr 14, 2019

コンフリクト解消

@syuilo
Copy link
Member

syuilo commented Apr 17, 2019

設定とか無しで、強制的に日付形式でも良さそう

@tamaina
Copy link
Contributor Author

tamaina commented Apr 17, 2019

えっなぜ

@syuilo
Copy link
Member

syuilo commented Apr 17, 2019

実装を複雑にしてまで設定できるようにする必要性あるかなーと思った

@rinsuki
Copy link
Contributor

rinsuki commented Apr 17, 2019

無意味なファイル名付いたファイル大量生産と日時で付けられたファイル名付いたファイル大量生産で前者を選択可能にする意味なさそう

@tamaina
Copy link
Contributor Author

tamaina commented Apr 17, 2019

あー

@tamaina
Copy link
Contributor Author

tamaina commented Apr 17, 2019

日付のフォーマットの指定のことかと思った
なんでそんなところまでオプションにしたか覚えていない……()

@syuilo
Copy link
Member

syuilo commented Apr 17, 2019

フォーマットに関しても設定できる必要性は薄そう

@rinsuki
Copy link
Contributor

rinsuki commented Apr 17, 2019

それはmmddyyyyじゃないとだめとかddmmyyyyじゃないとだめとかそういういちゃもんを付ける人がいそうなのであってもよさそう

@syuilo
Copy link
Member

syuilo commented Apr 17, 2019

ほむん

@tamaina
Copy link
Contributor Author

tamaina commented Apr 17, 2019

ん？すみません、もう一回聞きたいんですが、

AyaMorisawaの

Ctrl+V押した時点で名前を付けられるようになってほしい

が要らない感じですか？

@syuilo
Copy link
Member

syuilo commented Apr 17, 2019

うーむ要らなそう

@tamaina
Copy link
Contributor Author

tamaina commented Apr 17, 2019

私は要る

@mei23
Copy link
Contributor

mei23 commented Apr 17, 2019

私はルートフォルダにさえなければファイル名はなんでもいいわ

@tamaina tamaina changed the title Improve paste uploading Resolve #3023 WIP: Improve paste uploading Resolve #3023 Apr 17, 2019
@tamaina tamaina changed the title WIP: Improve paste uploading Resolve #3023 Improve paste uploading Resolve #3023 Apr 17, 2019
@tamaina
Copy link
Contributor Author

tamaina commented May 27, 2019

マージ不可orz

@tamaina tamaina closed this May 27, 2019
@tamaina
Copy link
Contributor Author

tamaina commented Jun 23, 2019

復帰

@tamaina tamaina reopened this Jun 23, 2019
@u1-liquid
Copy link
Sponsor Contributor

u1-liquid commented Jul 3, 2019 •
edited
Loading

https://github.com/u1-liquid/misskey/tree/improve-paste-uploading
Rebaseしてみた

https://suki.tsuki.network/ で動作確認済み

@syuilo
Copy link
Member

syuilo commented Jul 7, 2019

momentをimportするとスクリプトサイズ結構増えちゃう記憶があるのでなんとかmoment無しで実現できないかな

@acid-chicken
Copy link
Member

もう moment 混入してると思います。

@syuilo
Copy link
Member

syuilo commented Jul 7, 2019

今importしてるmomentは非同期コンポーネントなのでほとんど問題ないです

@u1-liquid
Copy link
Sponsor Contributor

momentをimportするとスクリプトサイズ結構増えちゃう

できるならnative tsにしてできないならday.jsとか使ってみる

@syuilo
Copy link
Member

syuilo commented Jul 7, 2019

このPRで編集されている投稿フォームウィジェットは非同期じゃなかったと思うのでそこを非同期にする方法もありそうですけど面倒そう

@syuilo
Copy link
Member

syuilo commented Jul 7, 2019

とりあえず既定のアップロード先フォルダ設定機能はmomentとは関係ないのでこのPRから先に抜き出して適用しようと思います

@syuilo
Copy link
Member

syuilo commented Jul 7, 2019

テンプレート設定はできないようにして、new Date().toLocaleString()の値を使うようにするという手もありそう

@u1-liquid
Copy link
Sponsor Contributor

せっかくだし、これのAuthorの @tamaina の意見を反映してまずどれかカスタマイズできるようにした方が良さそうだけど、moment.js なしでは day.js みたいな代替ライブラリを使わない限りある程度複雑になるのは事実なので、いっそのこと

{{year}}-{{month}}-{{day}} {{hour}}:{minute}}:{{second}} {{number}}

みたいにするのはどうかな

@syuilo
Copy link
Member

syuilo commented Jul 7, 2019

よさそう

@u1-liquid
Copy link
Sponsor Contributor

一旦 rebase した
もう適用されてる 841c8d6 の修正は排除しました

@u1-liquid
Copy link
Sponsor Contributor

image
image
作ってしまった・・・

@tamaina
Copy link
Contributor Author

tamaina commented Jul 8, 2019 •
edited
Loading

poll-editorにmomentが入ってるのでそれも非同期にしなきゃだめそう
https://github.com/syuilo/misskey/blob/develop/src/client/app/common/views/components/poll-editor.vue

@syuilo
Copy link
Member

syuilo commented Jul 8, 2019

poll-editorは非同期ですね

@tamaina
Copy link
Contributor Author

tamaina commented Jul 8, 2019

poll-editor自体が必ず非同期なんですねー

@syuilo syuilo merged commit 5343b00 into misskey-dev:develop Jul 8, 2019
@syuilo
Copy link
Member

syuilo commented Jul 8, 2019

ありがとうございます
🙏🙏🙏

@u1-liquid
Copy link
Sponsor Contributor

あ～～～～～バクひとつ遅れた⤵⤵⤵⤵⤵⤵

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@u1-liquid u1-liquid u1-liquid left review comments

@acid-chicken acid-chicken acid-chicken approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

投稿フォームにCtrl+Vでアップロードした画像に判別しやすい名前を付けてほしい
6 participants
@tamaina @syuilo @rinsuki @mei23 @u1-liquid @acid-chicken