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

Fix title in container #61

Merged
merged 2 commits into from
Dec 3, 2018
Merged

Fix title in container #61

merged 2 commits into from
Dec 3, 2018

Conversation

Takumi0901
Copy link
Contributor

issue

#18

上記issueが未対応のようだったので修正してみました。もしすでに対応されていましたらごめんなさい。

summary

Adjusted the layout collapse of title on smartphone view using negative margin

negative marginを使ってスマートフォンビューのtitleのレイアウト崩れを調整しました。

@BcRikko BcRikko self-requested a review December 3, 2018 05:45
@BcRikko BcRikko added the bug Something isn't working label Dec 3, 2018
@BcRikko
Copy link
Member

BcRikko commented Dec 3, 2018

ありがとうございます 🎉

ネガティブマージンでの対応、気づきませんでした 😅
いい感じです 👍
2018-12-03 14 47 33

ただwindow幅を広げたときに、タイトル部分も伸びてしまうようでした 😭
nes_css_-_nes-style_css_framework

@Takumi0901
Copy link
Contributor Author

@BcRikko
失礼しました汗

修正分をpushしましたのでご確認ください。
それに伴い.container.with-titleflex周りのスタイルを消去しています。私の方でも確認したところ他に影響はなさそうですが、念の為ご報告です。

@BcRikko
Copy link
Member

BcRikko commented Dec 3, 2018

早速対応いただき、ありがとうございます 🙇
LGTMです 🎉

flexboxは.container.with-title.is-center(.is-right)のようにしたときに、タイトルを中央寄せ、右寄せするためのものだったのですが、もとからバグっていたようですね 😇 ( #62 )
https://github.com/BcRikko/NES.css/blob/ceb018d8d6d6ad2d5a3c9937380261d915d1e275/scss/elements/containers.scss#L90-L96

@BcRikko BcRikko merged commit 51d748a into nostalgic-css:master Dec 3, 2018
@BcRikko
Copy link
Member

BcRikko commented Dec 18, 2018

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants