Skip to content

Validate cursorColor value and apply default if it fails #590

Merged
leo merged 1 commit intovercel:masterfrom
jwhitmarsh:fix/527
Aug 18, 2016
Merged

Validate cursorColor value and apply default if it fails #590
leo merged 1 commit intovercel:masterfrom
jwhitmarsh:fix/527

Conversation

@jwhitmarsh
Copy link
Contributor

@jwhitmarsh jwhitmarsh commented Aug 8, 2016

Originally was #562 but I did the work on master, so switched it to a branch. Apologies for the mess!

Fixes 527

@leo
Copy link
Contributor

leo commented Aug 11, 2016

👌 Please rebase.

@leo leo self-assigned this Aug 11, 2016
@leo leo changed the title Fix/527 Validate cursorColor value and apply default if it fails Aug 11, 2016
@jwhitmarsh
Copy link
Contributor Author

jwhitmarsh commented Aug 18, 2016

Was that rebase correct? Is this good to be merged now?

@MrRio
Copy link
Contributor

MrRio commented Aug 18, 2016

@jwhitmarsh I'm getting "This branch is out-of-date with the base branch - Merge the latest changes from master into this branch."

refactor `validateColor`: remove unnecessary variable declaration
@jwhitmarsh
Copy link
Contributor Author

Sorry, it was showing up as 'Good to merge' for me - i've re-rebased it :)

@leo leo merged commit 125fdd9 into vercel:master Aug 18, 2016
@leo
Copy link
Contributor

leo commented Aug 18, 2016

Thanks a lot! 😊

chabou added a commit to chabou/hyper that referenced this pull request Aug 19, 2016
* master:
  chore(package): update react to version 15.3.1 (vercel#637)
  Fix vercel#527: validate cursorColor value and apply default if it fails (vercel#590)
  Added customChildrenBefore to the tabs. (vercel#580)
  Fix for markdown files (vercel#618)
  Provide clear selection of text in terminal view (vercel#608)
  Added shellArgs to the config. (vercel#572)
  Fix international tilde character, and ` and ´ (vercel#584)
  chore(package): update electron-prebuilt to version 1.3.3 (vercel#604)
  chore(package): update should to version 11.0.0 (vercel#602)
  Comments for ignored stuff
  A little shorter
  Unneeded space
  Open new window offset from last focused window (Credit: albinekb) (vercel#581)
  Fix maximizing behaviour (vercel#176)
  Fix mapXDispatch and allow plugin to access onWheel (credit: lkzhao) (vercel#578)
  Use single quotes for better compatibility (vercel#575)
  Add config for bell (vercel#468)
  chore(package): update electron-prebuilt to version 1.3.2 (vercel#553)
  Moved "file-uri-to-path" dep to app package.json (vercel#569)
@timothyis timothyis added this to the v0.8.0 milestone Aug 29, 2016
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.

4 participants