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

ホームページのリニューアル #16

Merged
merged 2 commits into from
Jan 18, 2022
Merged

Conversation

pklion
Copy link
Contributor

@pklion pklion commented Jan 18, 2022

パスワード解析に参加できるほどの技量がないので、せめて貢献できればとホームページをリニューアルさせていただこうかと思っております。

現在の変更箇所

  • CSSの追加(スマホ対応)
  • zip, txt, sha256のaタグにdownload属性を追加 (もし不要でしたら削除します)

その他のコンテンツ追加等、案もあれば是非教えてください
力不足ですが対応できることがあれば、対応したいと思います
(多忙のため返信遅くなるかもしれません。すみません…)
コントリビューターの欄を更新したらDraftでなくそうと思うのですが、いかがでしょうか?

追記:index.html内で文字コードを指定したので、utf8版を削除したのですが、もし必要であれば追加したいと思います。
変更の確認はこちらから: https://pklion.github.io/yokai-password/website/

@kmikage
Copy link
Owner

kmikage commented Jan 18, 2022

@pklion
PRありがとうございます、確認させて頂きました。
mergeにあたり、文字コード周りの修正が必要なので、お時間いただければ幸いです。

@kmikage
Copy link
Owner

kmikage commented Jan 18, 2022

@pklion

実を言うと、レンタルサーバーの仕様上、SJISでアップロードが必要だったのですが、
実際の制作はUTF-8で行った方がラクだったので、アップロード用のスクリプト内でUTF-8からSJISに変換させていました。

cat index.utf8.html | nkf -s > index.html

あと、できればPRではなく、事前にIssueかDiscord Server内で相談頂ければ嬉しかったです、、 ><。。

@kmikage kmikage marked this pull request as ready for review January 18, 2022 11:47
@kmikage kmikage changed the base branch from main to issues/17 January 18, 2022 11:53
@kmikage kmikage merged commit 462ba83 into kmikage:issues/17 Jan 18, 2022
@kmikage
Copy link
Owner

kmikage commented Jan 18, 2022

#17 として、新しくbranchを切り、mergeしました。
以降は、 #17 として、issues/17にPR頂けると幸いです。

kmikage added a commit that referenced this pull request Jan 19, 2022
This reverts commit 462ba83, reversing
changes made to 7707934.
pklion added a commit to pklion/yokai-password that referenced this pull request Jan 19, 2022
commit 4ae3200
Author: kei <kmikage@gmail.com>
Date:   Wed Jan 19 18:17:53 2022 +0900

    Revert "Merge pull request kmikage#16 from pklion/upd-hp to issues/17."

    This reverts commit 462ba83, reversing
    changes made to 7707934.

commit abc3131
Merge: 6f86f56 dc2adb6
Author: Kei Mikage <kmikage@gmail.com>
Date:   Wed Jan 19 17:57:42 2022 +0900

    Merge pull request kmikage#19 from kmikage/revert-18-upd-hp

    Revert "Update website kmikage#2"

commit dc2adb6
Author: Kei Mikage <kmikage@gmail.com>
Date:   Wed Jan 19 17:56:57 2022 +0900

    Revert "Update website kmikage#2"

commit 6f86f56
Merge: 462ba83 019e25a
Author: Kei Mikage <kmikage@gmail.com>
Date:   Wed Jan 19 17:56:12 2022 +0900

    Merge pull request kmikage#18 from pklion/upd-hp

    Update website kmikage#2

commit 462ba83
Merge: 7707934 9607009
Author: Kei Mikage <kmikage@gmail.com>
Date:   Tue Jan 18 20:55:11 2022 +0900

    Merge pull request kmikage#16 from pklion/upd-hp to issues/17.

    issues/17にmerge.
    kmikage#17 として引き続き対応。
@pklion pklion mentioned this pull request Jan 20, 2022
3 tasks
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.

None yet

2 participants