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

screen-common.less: Use 'word-break: keep-all' #113

Merged
merged 1 commit into from Dec 22, 2019

Conversation

simnalamburt
Copy link
Member

@simnalamburt simnalamburt commented Dec 22, 2019

@simnalamburt simnalamburt self-assigned this Dec 22, 2019
@simnalamburt simnalamburt marked this pull request as ready for review December 22, 2019 16:21
@simnalamburt
Copy link
Member Author

@lens0021 screen-common.less 에 놓으면 모든 상황에서 (데스크톱 스크린, 모바일 스크린, 인쇄, ...) 항상 적용되는게 맞나요? 아니면 screen-common.less와 print.css 양쪽 모두에 추가해야하나요?

@lens0021
Copy link
Member

@lens0021 screen-common.less 에 놓으면 모든 상황에서 (데스크톱 스크린, 모바일 스크린, 인쇄, ...) 항상 적용되는게 맞나요? 아니면 screen-common.less와 print.css 양쪽 모두에 추가해야하나요?

@simnalamburt

  • print.css는 "…" 메뉴의 "인쇄"를 누르면 나타나는 인쇄 레이아웃 혹은 링크 https://femiwiki.com/index.php?title=아스트리드_린드그렌&printable=yes에서 사용되는 CSS입니다.
  • screen-common.less는 위에 언급한 인쇄 레이아웃을 제외하곤 모바일과 데스크톱 통틀어 거의 모든 상황에 적용되는 것이 맞는 것으로 알고 있습니다.

두 파일은 (일종의 매니페스트인) skin.json 파일에서 https://github.com/femiwiki/FemiwikiSkin/blob/d78f9f6/skin.json#L39-L50 와 같이 참조되고 있으므로, 더 자세한 정보는 https://www.mediawiki.org/wiki/Manual:Extension.json/Schema 문서를 참고해 주세요.

@simnalamburt
Copy link
Member Author

프린트될 때에도 단어 강제개행이 일어나질 원치 않으므로 print.css에도 동일한 수정을 넣겠습니다.

Copy link
Member

@lens0021 lens0021 left a comment

Choose a reason for hiding this comment

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

LGTM

@simnalamburt simnalamburt merged commit 0b21782 into master Dec 22, 2019
@simnalamburt simnalamburt deleted the simnalamburt-patch-1 branch December 22, 2019 16:45
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