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

印刷時にアトリビューションが表示されるように CSS の修正などをした #386

Merged
merged 2 commits into from
Oct 12, 2023
Merged

印刷時にアトリビューションが表示されるように CSS の修正などをした #386

merged 2 commits into from
Oct 12, 2023
+26 −0

Conversation

kamataryo
Copy link
Contributor

@kamataryo kamataryo commented Oct 11, 2023 •
edited
Loading

  • 印刷時は不要なボタン類を非表示にする
  • i アイコンでトグルされている時にアトリビューションが印刷されないのに対応。印刷のメディアに切り替わったときに、トグルを強制的に開く
@media screen @media print
スクリーンショット 2023-10-11 18 19 43 スクリーンショット 2023-10-11 18 19 51

close #384

@kamataryo kamataryo force-pushed the printing-style-sheet branch from 03c6b38 to 548be24 Compare October 11, 2023 08:57
@kamataryo kamataryo changed the title Add @media print CSS to remove unnecessary html element 印刷時にアトリビューションが表示されるように CSS を追加 Oct 11, 2023
phanect
phanect approved these changes Oct 11, 2023
View reviewed changes
Copy link
Contributor

@phanect phanect left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@kamataryo kamataryo changed the title 印刷時にアトリビューションが表示されるように CSS を追加 印刷時にアトリビューションが表示されるように CSS の修正などをした Oct 11, 2023
@naogify
Copy link
Member

naogify commented Oct 12, 2023

@kamataryo
ありがとうございます! 良いと思います。

ちなみに今回の修正と関係ないのですが、command + p で、印刷ダイアログを立ち上げた時に、自分の環境だと地図の右端に白い余白が入っていました。（開発者ツールで Emulate すると表示されないのですが）

修正前の環境でも表示されたので、ページの CSS の問題かもしれないです。

スクリーンショット 2023-10-12 10 59 28

https://github.com/geolonia/embed/files/12876199/%40geolonia_embed.pdf

keichan34
keichan34 approved these changes Oct 12, 2023
View reviewed changes
Copy link
Member

@keichan34 keichan34 left a comment

Choose a reason for hiding this comment

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

ありがとうございました！

@kamataryo
Copy link
Contributor Author

@naogify ありがとうございます。
ページ幅を変えるとなりがちで、多分 print メディアに切り替わると時に際レンダリングがされないとか、そういった maplibre 側の問題のように思います。

スクリーンショット 2023-10-12 11 57 57

naogify
naogify approved these changes Oct 12, 2023
View reviewed changes
Copy link
Member

@naogify naogify left a comment

Choose a reason for hiding this comment

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

@kamataryo
了解しました。ありがとうございます!

@kamataryo kamataryo merged commit 61ec0e8 into master Oct 12, 2023
@kamataryo kamataryo deleted the printing-style-sheet branch October 12, 2023 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@naogify naogify naogify approved these changes

@keichan34 keichan34 keichan34 approved these changes

@phanect phanect phanect approved these changes

@sugama-satsuki sugama-satsuki Awaiting requested review from sugama-satsuki

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

Successfully merging this pull request may close these issues.

印刷時のスタイルシートの改善
4 participants
@kamataryo @naogify @phanect @keichan34