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

GUI fixes #676

Merged
merged 62 commits into from
Mar 29, 2024
Merged

GUI fixes #676

merged 62 commits into from
Mar 29, 2024

Conversation

anatawa12
Copy link
Member

@anatawa12 anatawa12 commented Mar 28, 2024

Fixes #617
Fixes #622
Fixes #623
Fixes #638
Fixes #640
Fixes #647
Fixes #646
Fixes #644
Fixes #670
Fixes #650
Fixes #652
Fixes #669
Fixes #645
Fixes #639
Fixes #642
Fixes #681

@Sayamame-beans
Copy link
Collaborator

今気付いたのですが、ワールドプロジェクト(3.4.2, 3.5.2)においてUdonSharpが追加可能なパッケージとして表示されています
いつからのバグでしょう…?

@anatawa12
Copy link
Member Author

今気付いたのですが、ワールドプロジェクト(3.4.2, 3.5.2)においてUdonSharpが追加可能なパッケージとして表示されています
いつからのバグでしょう…?

気にしてなかった

@Sayamame-beans
Copy link
Collaborator

ClientSimとUdonSharpは3.4.0以降が入っているなら表示されないというのがVCCの挙動ですが、どうします…? 一覧には出しておいて、3.4.0以降が入っている場合は、互換性なし扱いの表示/ダイアログ表示にするとかの方がanatawaさん好みの挙動です?

@anatawa12
Copy link
Member Author

入ってるやつのlegacy packagesは非表示にするのがまぁ妥当なのはそうなのでそうします

@anatawa12
Copy link
Member Author

プロジェクトを削除しようとした時に出るダイアログのプロジェクト名部分です

は謎だけど解決策がわからんのでfont-normalを追加します

@anatawa12
Copy link
Member Author

多分ok

@Sayamame-beans
Copy link
Collaborator

一覧から見えなくなるの動いたので、ここまでで挙げた問題は軒並み解決したと思います。
…あ、Changelogに載せないとですね。

これだけちょっと気になりましたが、これはvrc-get-vpmの問題っぽい気もする? (今回はスルーしても良さそう)
image

@Sayamame-beans
Copy link
Collaborator

載せるとしたら、空白表示問題と、末尾空白問題と、Unity追加失敗問題と、レガシーパッケージ表示についての4つ?

@anatawa12
Copy link
Member Author

レガシーが競合するのは直さないとですね。vpmなので両方に書く

CHANGELOG-gui.md Show resolved Hide resolved
@Sayamame-beans
Copy link
Collaborator

並び順そうなるんだの顔してます(全く致命的ではない(全部読めば良いだけなので))
image

@Sayamame-beans
Copy link
Collaborator

ちなみに逆になったりもします
(きっとここの表示順は固定じゃないんでしょうね)

@anatawa12
Copy link
Member Author

そこの並び順、何もルールがないので完全に不定ですね。hashmapなので

Copy link
Collaborator

@Sayamame-beans Sayamame-beans left a comment

Choose a reason for hiding this comment

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

気になるところはありますが、とりあえず自然言語的にはOKだと思います

vrc-get-gui/locales/en.json5 Outdated Show resolved Hide resolved
@Sayamame-beans
Copy link
Collaborator

そこの並び順、何もルールがないので完全に不定ですね。hashmapなので

何となく、非決定的な中でもインストールとアンインストールぐらいの単位では表示が分かれてると嬉しそうですが、私だけかもしれませんし、このPRの範疇でもなさそうかな

@Sayamame-beans
Copy link
Collaborator

気になるところはありますが、とりあえず自然言語的にはOKだと思います

あー、boldになった項目増えてるから大丈夫じゃなかった

Copy link
Collaborator

@Sayamame-beans Sayamame-beans left a comment

Choose a reason for hiding this comment

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

last review

CHANGELOG-gui.md Outdated Show resolved Hide resolved
vrc-get-gui/locales/en.json5 Outdated Show resolved Hide resolved
vrc-get-gui/locales/ja.json5 Outdated Show resolved Hide resolved
vrc-get-gui/lib/remove-project.tsx Outdated Show resolved Hide resolved
Copy link
Collaborator

@Sayamame-beans Sayamame-beans left a comment

Choose a reason for hiding this comment

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

LGTM! Really excellent work!

@anatawa12 anatawa12 merged commit 7bd05be into master Mar 29, 2024
17 checks passed
@anatawa12 anatawa12 deleted the fix-gui branch March 29, 2024 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment