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

CoMETIKメンバー追加 #543

Merged
merged 6 commits into from
Jan 6, 2024
Merged

CoMETIKメンバー追加 #543

merged 6 commits into from
Jan 6, 2024

Conversation

gomao9
Copy link
Contributor

@gomao9 gomao9 commented Aug 2, 2023

まだプロフィール情報が出きってませんが取り急ぎ。

@gomao9 gomao9 changed the title Cometikメンバー追加 Draft: Cometikメンバー追加 Aug 2, 2023
@gomao9 gomao9 marked this pull request as draft August 2, 2023 04:44
@gomao9 gomao9 changed the title Draft: Cometikメンバー追加 Cometikメンバー追加 Aug 2, 2023
@gomao9 gomao9 changed the title Cometikメンバー追加 CoMETIKメンバー追加 Aug 2, 2023
@gomao9 gomao9 marked this pull request as ready for review August 2, 2023 04:59
@gomao9 gomao9 requested a review from a team August 3, 2023 14:03
Copy link
Member

@banjun banjun left a comment

Choose a reason for hiding this comment

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

内容(リテラル)確認しました。あってそうです。

内容付き合わせは下記ソースで
https://shinycolors-song-for-prism.idolmaster-official.jp/idol/#com

アイドルのカラーもここから拾えるんですね(このカラーコードと一緒なことを確認しました)

@gomao9 gomao9 removed the request for review from a team August 3, 2023 22:00
@gomao9
Copy link
Contributor Author

gomao9 commented Aug 3, 2023

レビューありがとうございます!助かります。

@gomao9
Copy link
Contributor Author

gomao9 commented Aug 8, 2023

レビュー通りましたので問題なければマージ頂ければと。
マージ権あるのは @imas/imasparql の方でしょうか。よろしくお願いします。

RDFs/283.rdf Outdated
<imas:cv rdf:resource="https://ja.dbpedia.org/page/三川華月"/>
<imas:cv rdf:resource="https://www.wikidata.org/wiki/Q107587275"/>
<imas:Color rdf:datatype="http://www.w3.org/2001/XMLSchema#hexBinary">E0B5D3</imas:Color>
<schema:birthPlace xml:lang="ja">長野県</schema:birthPlace>
Copy link
Member

Choose a reason for hiding this comment

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

岡山県

Copy link
Member

Choose a reason for hiding this comment

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

見逃してた...あざます

Copy link
Contributor Author

Choose a reason for hiding this comment

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

危ない、ありがとうございます。修正しました。

Copy link
Member

@crssnky crssnky left a comment

Choose a reason for hiding this comment

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

色についてですが、どこから採取しているでしょうか?

@crssnky
Copy link
Member

crssnky commented Aug 14, 2023

たいへん遅くなり申し訳ございませんがよろしくお願いします!

@banjun
Copy link
Member

banjun commented Aug 14, 2023

色これっぽいけど,既存の色採取のものとは違ってそう #543 (review)

image

@crssnky
Copy link
Member

crssnky commented Aug 14, 2023

image
image
一応、文字の色がそうかなと思って採取したもの(1枚目)とPulReqの色(2枚目)を貼ってみました。
まだ文字の色がそうとうも決まっていないので正解不正解はわかりませんが参考までに。

@crssnky
Copy link
Member

crssnky commented Aug 14, 2023

image
もしやと思い、樹里の文字色を調べてみました。
enza版のアイドル紹介の枠線と同じ値を示していたので、文字色を正としても良いかと思います
訂正:千雪さんのシャニソンの紹介を見ると真っ白では無いので、明るすぎる場合は暗くしている可能性があります

@gomao9
Copy link
Contributor Author

gomao9 commented Aug 19, 2023

実際は背景からカラーピッカーでとったんですが、要するに上述のshare.cssで定義されている色を取っています。
登録済みアイドルの色が一致しているので良さそうかな~と思っています。

.idol-modal__item.hiori .idol-modal__bg-square   { background-color: #144384 }
.idol-modal__item.mano .idol-modal__bg-square    { background-color: #ffbad6 }
.idol-modal__item.meguru .idol-modal__bg-square  { background-color: #ffe012 }
.idol-modal__item.mamimi .idol-modal__bg-square  { background-color: #a846fb }
.idol-modal__item.kiriko .idol-modal__bg-square  { background-color: #b3e4ff }
.idol-modal__item.kogane .idol-modal__bg-square  { background-color: #f84cad }
.idol-modal__item.yuika .idol-modal__bg-square   { background-color: #3b91c4 }
.idol-modal__item.sakuya .idol-modal__bg-square  { background-color: #006047 }
.idol-modal__item.rinze .idol-modal__bg-square   { background-color: #89c3eb }
.idol-modal__item.chiyoko .idol-modal__bg-square { background-color: #f93b90 }
.idol-modal__item.kaho .idol-modal__bg-square    { background-color: #e5461c }
.idol-modal__item.juri .idol-modal__bg-square    { background-color: #ffc602 }
.idol-modal__item.natsuha .idol-modal__bg-square { background-color: #90e667 }
.idol-modal__item.amana .idol-modal__bg-square   { background-color: #f54275 }
.idol-modal__item.chiyuki .idol-modal__bg-square { background-color: #dbdbdb }
.idol-modal__item.tenka .idol-modal__bg-square   { background-color: #e75bec }
.idol-modal__item.fuyuko .idol-modal__bg-square  { background-color: #5aff19 }
.idol-modal__item.asahi .idol-modal__bg-square   { background-color: #f30100 }
.idol-modal__item.mei .idol-modal__bg-square     { background-color: #f0f }
.idol-modal__item.hinana .idol-modal__bg-square  { background-color: #ffc639 }
.idol-modal__item.toru .idol-modal__bg-square    { background-color: #50d0d0 }
.idol-modal__item.madoka .idol-modal__bg-square  { background-color: #be1e3e }
.idol-modal__item.koito .idol-modal__bg-square   { background-color: #7967c3 }
.idol-modal__item.nichika .idol-modal__bg-square { background-color: #a6cdb6 }
.idol-modal__item.mikoto .idol-modal__bg-square  { background-color: #760f10 }
.idol-modal__item.hana .idol-modal__bg-square    { background-color: #e0b5d3 }
.idol-modal__item.luca .idol-modal__bg-square    { background-color: #24130d }
.idol-modal__item.haruki .idol-modal__bg-square  { background-color: #ead7a4 }

@gomao9 gomao9 requested a review from crssnky August 19, 2023 10:42
@crssnky
Copy link
Member

crssnky commented Aug 27, 2023

霧子の値が#b3e4ffとなっていますが、im@sparqlが参考にしたenza版のアイドルの枠の色( https://shinycolors.idolmaster.jp/idol/lantica/kiriko.html )は#D9F2FFなんですよね...
同様に、千雪さんの#dbdbdb#FBFAFAと異なっています。
もしかしたら薄い色は濃い方向へ持ち上げられてるかもしれません。

@arrow2nd
Copy link
Member

enza版出てましたね…!

https://shinycolors.idolmaster.jp/idol/cometik/luca.html

Firefoxのスポイト機能で枠線の色を拾ったところ、ルカさんの色が一致したのでこちらの値でも良さそうに思いました。
シャニソンのものより若干濃い感じですかね…?

image
  • 鈴木羽那さん #e0b5d3
  • 斑鳩ルカさん #24130d
  • 郁田はるきさん #ead7a4

@banjun
Copy link
Member

banjun commented Nov 9, 2023

enza版来たならそれが良いのでは

@crssnky crssnky merged commit 7590b9b into imas:master Jan 6, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants