-
Notifications
You must be signed in to change notification settings - Fork 0
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
[#96] Use u-height-stretch instead of… #97
base: main
Are you sure you want to change the base?
[#96] Use u-height-stretch instead of… #97
Conversation
…u-height-100vh in sidebar #96
6000148
to
8a07290
Compare
@@ -71,7 +71,8 @@ defmodule BitstylesPhoenixDemo.MixProject do | |||
], | |||
bitstyles: [ | |||
"cmd mkdir -p priv/static/assets", | |||
"cmd cp -R assets/node_modules/bitstyles/assets/* priv/static/assets" | |||
"cmd cp -R assets/node_modules/bitstyles/assets/* priv/static/assets", | |||
"cmd cp -R ../assets/logo.svg priv/static/assets/images/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
side thing, since the demo app image is broken
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’ve left suggestions with the current classes we use on frontend apps. It might mean that we don't need the autoprefixer, as it contains a fallback already — let me check it locally this afternoon though
I just pushed a commit with updated classnames — it works for me in safari, firefox, and chrome, with or without autoprefixer (the extra niceness of I think the autoprefixer flag is unnecessary in this case, wdyt? |
…u-height-100vh in sidebar
#96