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

Color Schemes In Terminal #484

Closed

Conversation

samirmansour
Copy link
Contributor

This branch combines the ability to have sessions and the ability to change the terminals color scheme on an individual basis. To add color schemes go to ~/.config/ondemand/apps/shell/themes. To launch a new or existing terminal with a theme go to /launch endpoint.

@ericfranz
Copy link
Contributor

@samirmansour noted that the failing build is due to an npm dependency conflict where it is asking which version we want to install; the question is whether this is preexisting or introduced by this PR

apps/shell/app.js Outdated Show resolved Hide resolved
@ericfranz ericfranz changed the base branch from master to reconnection-improvements May 19, 2020 11:50
apps/shell/app.js Outdated Show resolved Hide resolved
apps/shell/app.js Outdated Show resolved Hide resolved
apps/shell/app.js Outdated Show resolved Hide resolved
})
}

function rgbToHexMath (num) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add some documentation above all of these functions following https://jsdoc.app/ format

@ericfranz ericfranz added this to the OOD1.8 milestone Jul 2, 2020
@ericfranz ericfranz modified the milestones: OOD1.8, OOD2.0 Aug 4, 2020
@ericfranz
Copy link
Contributor

I marked this as blocked and for 2.0 - as it is a more complex solution that piggybacks off of the re-connection improvements.

We are looking at a simpler solution to add sooner.

@ericfranz
Copy link
Contributor

Duplicate of #630

@ericfranz ericfranz marked this as a duplicate of #630 Feb 25, 2021
@ericfranz ericfranz closed this Feb 25, 2021
@ericfranz ericfranz deleted the term-color-improvements branch February 25, 2021 20:53
@ericfranz ericfranz removed this from the OOD2.0 milestone Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants