-
Notifications
You must be signed in to change notification settings - Fork 417
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
Add shutdown button #862
Add shutdown button #862
Conversation
lib/livebook_web/live/home_live.ex
Outdated
path={Routes.settings_path(@socket, :page)} | ||
active={false} /> | ||
<SidebarHelpers.user_item current_user={@current_user} path={Routes.home_path(@socket, :user)} /> | ||
<SidebarHelpers.shared_home_footer socket={@socket} current_user={@current_user} /> |
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.
Maybe default_footer
, since we mirror it everywhere except the session page?
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.
The naming is tricky. I am calling "home" all of the explorer, settings, etc. I think default is not very descriptive. :'( Any other ideas?
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.
shared_home_footer
-> shared_footer
🙈
I cannot think of a good word to describe all off-session pages :<
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.
Yeah, I will keep the naming as is, those are all home-like pages or accessible from home. Once we expand things maybe we will have better naming ideas.
The button only appears on interactive mode because we assume embedded mode is running inside a release or similar.
95f143f
to
9568d6f
Compare
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.
🚢 !!
💚 💙 💜 💛 ❤️ |
The button only appears on interactive mode because
we assume embedded mode is running inside a release
or similar.