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

Use parameter for nocolor #126

Closed
esetera opened this issue Aug 26, 2011 · 17 comments
Closed

Use parameter for nocolor #126

esetera opened this issue Aug 26, 2011 · 17 comments
Assignees
Milestone

Comments

@esetera
Copy link

esetera commented Aug 26, 2011

It would be good to have a parameter to turn off the colors by default

@bingjie2680
Copy link

at least should assign very light colour for authors, because dark colour causes readability issues.

@ghost ghost assigned JohnMcLear Sep 1, 2011
@JohnMcLear
Copy link
Member

Do you want the author to not be able to see their colour AND do you want the viewer to not be able to see?

Do you want the user to be able to toggle this value or do you want this value to be set at a "pad" level?

@JohnMcLear
Copy link
Member

Currently I'm thinking I will pass &noColor=true and it will just hide all backgrounds colors. Should be easy"ish" to do.. Will take a look

@jeff-h
Copy link

jeff-h commented Sep 6, 2011

I hadn't realised this issue was here, so put something very similar at t https://github.com/Pita/etherpad-lite/issues/151#issuecomment-1989531 — my personal preference / use case would be for nobody to see colours just like you've implemented... thanks!

However perhaps a nice feature would be to allow each user to decide for themselves whether they will have a colour, or no colour, on a personal basis. Thus you could had a colour, and I could have none, and I would see your edits in your colour. I am sure that's more complex to code though! :)

@JohnMcLear
Copy link
Member

Well the ?noColor param is afaik per user :)

@jeff-h
Copy link

jeff-h commented Sep 6, 2011

Sorry, I meant per user as in I could have a colour, and you could have none, and when we each look at our respective browser's copies of the document, we will both see my edits in my colour, and yours in none.

@JohnMcLear
Copy link
Member

so I understand this as "Everyone else has no color". Will take a goosey.

@jhollinger
Copy link
Contributor

I would expect a noColor param to simply "hide" the colors in that window only. Everyone else would see the colors, including the color of the person who's hiding the colors. In other words it's just a view option. Actually it might make a nice Hide Authorship Colors button - an alternative to Clear Authorship Colors?

@JohnMcLear
Copy link
Member

You can do this with a parameter. ?noColors=true

-----Original Message-----
From: Jordan Hollinger [mailto:reply@reply.github.com]
Sent: 04 December 2011 18:02
To: John McLear
Subject: Re: [etherpad-lite] Use parameter for nocolor (#126)

I would expect a noColor param to simply "hide" the colors in that window only. Everyone else would see the colors, including the color of the person who's hiding the colors. In other words it's just a view option. Actually it might make a nice Hide Authorship Colors button - an alternative to Clear Authorship Colors?


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/issues/126#issuecomment-3007768
This email and its attachments may be confidential and are intended solely for the use of the individual to whom it is addressed. Any views or opinions expressed are solely those of the author and do not necessarily represent those of the organisation from which this email originated. If you are not the intended recipient of this email and its attachments, you must take no action based upon them, nor must you copy or show them to anyone. Please contact the sender if you believe you have received this email in error. This email was sent by School Email - Safe Webmail and Hosted Email for Schools

@Pita
Copy link
Contributor

Pita commented Dec 4, 2011

everyone else has no color just confuses the user and has no benefit at all imho

@jhollinger
Copy link
Contributor

IMHO, the existing noColor option is sufficient. Sure, there are plenty of other color/no-color options we could imagine, but are they useful enough to outweigh the potential ambiguity and confusion?

@JohnMcLear
Copy link
Member

Just to add my 2 pence as the person doing the dev on this. Actually making this change is proving to be pretty difficult, not sure if I'm missing something..

I know where and what to do but it feels dirty.

-----Original Message-----
From: Jordan Hollinger [mailto:reply@reply.github.com]
Sent: 04 December 2011 19:04
To: John McLear
Subject: Re: [etherpad-lite] Use parameter for nocolor (#126)

IMHO, the existing noColor option is sufficient. Sure, there are plenty of other color/no-color options we could imagine, but are they useful enough to outweigh the potential ambiguity and confusion?


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/issues/126#issuecomment-3008109
This email and its attachments may be confidential and are intended solely for the use of the individual to whom it is addressed. Any views or opinions expressed are solely those of the author and do not necessarily represent those of the organisation from which this email originated. If you are not the intended recipient of this email and its attachments, you must take no action based upon them, nor must you copy or show them to anyone. Please contact the sender if you believe you have received this email in error. This email was sent by School Email - Safe Webmail and Hosted Email for Schools

@JohnMcLear
Copy link
Member

I just pushed this back to 1.2

@marcelklehr
Copy link
Contributor

close this, pls.

@JohnMcLear
Copy link
Member

I'm not sure this can be closed? Afaik they want different functionality.

@marcelklehr
Copy link
Contributor

yeah, but that's not the subject of the issue, here...

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

No branches or pull requests

7 participants