Skip to content

Switch screen functionality #4

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

Merged
merged 1 commit into from
Apr 23, 2018
Merged

Switch screen functionality #4

merged 1 commit into from
Apr 23, 2018

Conversation

mikeymike
Copy link
Member

There's 2 "screens" that I've called main and alternate here
Switching the screen allows doing things without impact the main one

vim and less use this which is why when you quit you have no evidance of it


I was debating about the api for this, switchScreen and track internally or have two very specific methods and zero internal tracking. Let me know what you think

@mikeymike mikeymike requested a review from AydinHassan April 22, 2018 22:06
@codecov-io
Copy link

codecov-io commented Apr 22, 2018

Codecov Report

Merging #4 into master will increase coverage by 0.48%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master       #4      +/-   ##
============================================
+ Coverage     80.92%   81.41%   +0.48%     
- Complexity       80       82       +2     
============================================
  Files             7        7              
  Lines           152      156       +4     
============================================
+ Hits            123      127       +4     
  Misses           29       29
Impacted Files Coverage Δ Complexity Δ
src/UnixTerminal.php 71.42% <100%> (+1.56%) 41 <2> (+2) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c3a87b9...78b2f18. Read the comment docs.

@AydinHassan
Copy link
Member

@mikeymike I definitely prefer the approach in this PR - showPrimaryScreen() and showSecondaryScreen() is another option - your call 😊

There's 2 "screens" that I've called primary and secondary  here
Switching the screen allows doing things without impact the primary one

vim and less use this which is why when you quit you have no evidance of it
@mikeymike
Copy link
Member Author

hmm yes I like primary/secondary ... updated 😀

@AydinHassan AydinHassan merged commit 838d8d7 into master Apr 23, 2018
@AydinHassan AydinHassan deleted the switch-buffer branch April 23, 2018 08:09
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.

3 participants