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

“This is a live shell. Type in here.” gets old *really* fast. #315

Closed
freysie opened this issue Oct 24, 2015 · 4 comments · Fixed by #490
Closed

“This is a live shell. Type in here.” gets old *really* fast. #315

freysie opened this issue Oct 24, 2015 · 4 comments · Fixed by #490
Milestone

Comments

@freysie
Copy link

freysie commented Oct 24, 2015

We know it’s live already. No need to tell us.

Granted, this is nitpicking, but it would a lot cleaner without that message. Please? 😁

Apart from that little detail, I absolutely love this gem. Great job, guys!

@rstacruz
Copy link
Collaborator

on that note, it may be a good time to revisit the design entirely. @charliesome?

@lacostenycoder
Copy link

@alminde you could literally fix that in 30 seconds.
fork gem. find and remove or comment out these 3 lines of htmel code in variable_info.erb in templates.
push to your own repo and use that in your gemfile.
Done.

    <div class="hint">
        This is a live shell. Type in here.
    </div>

@freysie
Copy link
Author

freysie commented Nov 13, 2015

Yeah, @lacostenycoder. Thanks.

In fact I already did but just forgot to make a pull request.

@RobinDaugherty
Copy link
Member

I'm pretty sure the solution here is to remove the "helpful prompt" after the REPL has been used, rather than removing it forever. This could even be stored in a cookie or local storage so that the prompt isn't shown to the same developer twice (on the same project at least).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants