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

websiteにスタイルを適用 #17

Closed
kmikage opened this issue Jan 18, 2022 · 12 comments
Closed

websiteにスタイルを適用 #17

kmikage opened this issue Jan 18, 2022 · 12 comments

Comments

@kmikage
Copy link
Owner

kmikage commented Jan 18, 2022

#16 で、index.html へのCSS適用を頂いているので、mergeに向けて対応。
kmikage added a commit that referenced this issue Jan 18, 2022
@kmikage
Merge pull request #16 from pklion/upd-hp to issues/17.
462ba83 
issues/17にmerge.
#17 として引き続き対応。
@kmikage kmikage mentioned this issue Jan 18, 2022
Merged
@kmikage
Copy link
Owner Author

kmikage commented Jan 18, 2022 •
edited
Loading

修正が必要な箇所

  • ファイル名 ( index.html -> index.utf8.html )
  • 文字コードの指定の削除
  • （できれば）CSSは外部ファイルにして、importしてもらえると嬉しい。

ここまでの経緯

  • レンタルサーバー(XREA FreePlan)では、広告表示の兼ね合いでSJISでのアップロードが必須だった
  • 実際の制作そのものは、UTF-8で行い、アップロード用のスクリプト内で文字コードを変換していた
  • そのため、オリジナルのファイル名は、 index.utf8.html となっていた
  • websiteは誰も触らないと思い、手を抜いていた 笑

@pklion
Copy link
Contributor

pklion commented Jan 18, 2022

対応ありがとうございます。
修正の件について、了解です。
Githubの扱いにいまだ慣れていないので、変な操作などしてしまったらすみません…
CSSは別ファイルにして良いか分からなかったのでインラインにしておりました
すぐに変更しますね

@pklion
Copy link
Contributor

pklion commented Jan 18, 2022

すみません… issue/17 のブランチにcommitできないです…(アクセス許可がないのか、はたまた私の操作が間違っているのか…)
もし良ければ方法をご教授ください
(環境はGithub Desktop on Windows 11です)

@kmikage
Copy link
Owner Author

kmikage commented Jan 18, 2022

@pklion
了解です、修正よろしくお願いします。

GitHubについては、みんな通る道ですので 笑
のんびりいきましょうー

僕も明日・明後日と用事が重なっているので、レス遅くなるかも知れません。

@kmikage
Copy link
Owner Author

kmikage commented Jan 18, 2022 •
edited
Loading

@pklion

issue/17 のブランチにcommitできないです…

pklionさんのリポジトリにcommit, pushした後、先と同じようにPullRequestして下さいです。
先のPRでは、mainに対してのPRでしたが、issues/17宛にPRいただければ。

@pklion
Copy link
Contributor

pklion commented Jan 18, 2022

あ、なるほど。
了解しました。

@pklion pklion mentioned this issue Jan 18, 2022
Merged
@pklion
Copy link
Contributor

pklion commented Jan 18, 2022 •
edited
Loading

PR作成させていただきました。
Contributors の欄に抽出の方々を入れるのもアリだな、と思いつつリストがないので迷っているところです…いかがいたしましょう？

追記:目次もどこかで追加しようと思います…ゆったりですみません(^ ^；

@kmikage
Copy link
Owner Author

kmikage commented Jan 19, 2022

@pklion
PR #18 ですが、以下のコミットまででPRし直して頂けませんか？
33b553c

以降のコミットなのですが、mergeの前に、一度お話をする必要があるかと思います。
DiscordでDMさせて頂きますね。

@pklion
Copy link
Contributor

pklion commented Jan 19, 2022 •
edited
Loading

Revert してみたのですが如何でしょう？
ちょっとお待ちください…(操作が難しいのです)

pklion added a commit to pklion/yokai-password that referenced this issue Jan 19, 2022
@pklion
Squashed commit of the following:
7f4f8e3 
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
Copy link
Contributor

pklion commented Jan 19, 2022

お騒がせしました…これで多分戻っているはずです…

@pklion pklion mentioned this issue Jan 20, 2022
Closed
3 tasks
kmikage added a commit that referenced this issue Jan 21, 2022
@kmikage
Merge pull request #20 from pklion/upd-hp
7b95b1c 
Update website #17 .
@kmikage
Copy link
Owner Author

kmikage commented Jan 21, 2022

サイトの修正は、以上で終わり。
アップロード用のツール（非公開）を修正します。

@kmikage kmikage mentioned this issue Jan 21, 2022
Merged
kmikage added a commit that referenced this issue Jan 21, 2022
@kmikage
Merge pull request #23 from /issues/17
34d123a 
merge to main.
kmikage added a commit that referenced this issue Jan 21, 2022
@kmikage
Merge pull request #24 from /issues/17
090792c 
index.utf8.htmlの表記ミスを修正
@kmikage
Copy link
Owner Author

kmikage commented Jan 21, 2022

#24 で取り込みました。

@kmikage kmikage closed this as completed Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kmikage @pklion