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

Button for useMonospaceFont #128

Closed
wants to merge 3 commits into from
Closed

Button for useMonospaceFont #128

wants to merge 3 commits into from

Conversation

themasch
Copy link

as discussed in irc. Here it is!

@wscott
Copy link

wscott commented Sep 2, 2011

Not used? That is the one feature I was missing to make this useful.

Perhaps there is a config option to ALWAYS use monospace?
I did discover I can add ?useMonospacedFont=true to the URL to control this. Useful, but a bit awkward

@themasch
Copy link
Author

themasch commented Sep 2, 2011

@wscott: i absolutely second that!

@ggiammat
Copy link

ggiammat commented Sep 2, 2011

I'm missing this feature too!

@vaab
Copy link

vaab commented Sep 15, 2011

@wscott: I'm missing this feature too...

I got to the same point: using '?useMonospaceFont=true' works, but I would be glad to set this to ALWAYS use monospace.

@raphaelbastide
Copy link

I agree, The button should be a default checked "use monospace"

@waldyrious
Copy link

+1. An url parameter is useful but definitely not appropriate as an interface to toggle an option.

@Wikinaut
Copy link
Contributor

+1 , it is a perfect patch, I have tested it. Pita: please pull this in.

@Pita
Copy link
Contributor

Pita commented Nov 18, 2011

@johnyma22 In my opinion we should merge this

@JohnMcLear
Copy link
Member

It needs to have a setting in settings.json

-----Original Message-----
From: Peter 'Pita' Martischka [mailto:reply@reply.github.com]
Sent: 18 November 2011 22:39
To: John McLear
Subject: Re: [etherpad-lite] Button for useMonospaceFont (#128)

@johnyma22 In my opinion we should merge this


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2795307
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;/p>

@Pita
Copy link
Contributor

Pita commented Nov 19, 2011

@johnyma22 I don't agree with that. Whats your problem with the monospace button?

@JohnMcLear
Copy link
Member

My users don't want it :|

I could comment in out in CSS but the use case of monocase is pretty rare.

How many people that dont use linux do you think use monospace on a daily basis?


From: Peter 'Pita' Martischka [reply@reply.github.com]
Sent: 19 November 2011 00:52
To: John McLear
Subject: Re: [etherpad-lite] Button for useMonospaceFont (#128)

@johnyma22 I don't agree with that. Whats your problem with the monospace button?


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2796525
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;/p>

@Pita
Copy link
Contributor

Pita commented Nov 19, 2011

every programmer needs it... It's not a linux thing

On 18 November 2011 16:58, John McLear
reply@reply.github.com
wrote:

My users don't want it :|

I could comment in out in CSS but the use case of monocase is pretty rare.

How many people that dont use linux do you think use monospace on a daily basis?


From: Peter 'Pita' Martischka [reply@reply.github.com]
Sent: 19 November 2011 00:52
To: John McLear
Subject: Re: [etherpad-lite] Button for useMonospaceFont (#128)

@johnyma22 I don't agree with that. Whats your problem with the monospace button?


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2796525
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;/p>


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2796564

@Wikinaut
Copy link
Contributor

For example, when you post parts of program or when you add a table.

It would be a "nice to have" to allow users to set monospace per-character like bold, italics, underline, strike-trough, and not only per-pad.

@JohnMcLear
Copy link
Member

And < 1% of Etherpad users are programmers, I'm representing the 99% here, by default if a feature is only required by 1% of our user base then it shouldn't be enabled by default if it uses pixel space on the main screen.

-----Original Message-----
From: Peter 'Pita' Martischka [mailto:reply@reply.github.com]
Sent: 19 November 2011 01:36
To: John McLear
Subject: Re: [etherpad-lite] Button for useMonospaceFont (#128)

every programmer needs it... It's not a linux thing

On 18 November 2011 16:58, John McLear
reply@reply.github.com
wrote:

My users don't want it :|

I could comment in out in CSS but the use case of monocase is pretty rare.

How many people that dont use linux do you think use monospace on a daily basis?


From: Peter 'Pita' Martischka [reply@reply.github.com]
Sent: 19 November 2011 00:52
To: John McLear
Subject: Re: [etherpad-lite] Button for useMonospaceFont (#128)

@johnyma22 I don't agree with that. Whats your problem with the monospace button?


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2796525
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;/p>


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2796564


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2796793
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;/p>

@JohnMcLear
Copy link
Member

Also you can pass a parameter in the URL if you want userspace, if you say that it's useful for programmers I would like to think that programmers know how to pass a parameter via the URL ;)

-----Original Message-----
From: Peter 'Pita' Martischka [mailto:reply@reply.github.com]
Sent: 19 November 2011 01:36
To: John McLear
Subject: Re: [etherpad-lite] Button for useMonospaceFont (#128)

every programmer needs it... It's not a linux thing

On 18 November 2011 16:58, John McLear
reply@reply.github.com
wrote:

My users don't want it :|

I could comment in out in CSS but the use case of monocase is pretty rare.

How many people that dont use linux do you think use monospace on a daily basis?


From: Peter 'Pita' Martischka [reply@reply.github.com]
Sent: 19 November 2011 00:52
To: John McLear
Subject: Re: [etherpad-lite] Button for useMonospaceFont (#128)

@johnyma22 I don't agree with that. Whats your problem with the monospace button?


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2796525
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;/p>


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2796564


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2796793
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;/p>

@rspeer
Copy link
Contributor

rspeer commented Nov 20, 2011

Monospace is useful, especially for the kind of people who'll be running an experimental new version of EtherPad. Why not pull in the change, and then johnyma22 or whoever wants to can make the config option to hide it on certain sites?

@JohnMcLear
Copy link
Member

Displaying a monospace button should be "opt in" not "opt out", I know this from gathering stats across a large number of live Etherpad sites.

This isn't just based on opinion.

If anyone else has stats that argue otherwise feel free to contribute to the discussion.

-----Original Message-----
From: Rob Speer [mailto:reply@reply.github.com]
Sent: 20 November 2011 04:44
To: John McLear
Subject: Re: [etherpad-lite] Button for useMonospaceFont (#128)

Monospace is useful, especially for the kind of people who'll be running an experimental new version of EtherPad. Why not pull in the change, and then johnyma22 or whoever wants to can make the config option to hide it on certain sites?


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-2802514
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;/p>

@crash-dive
Copy link

Although I don't have any stats I am inclined to agree with johnyma22 on this. Although I can see why some people might want it, I do not think this is something the average user would need and therefore it should be off by default.

@vaab
Copy link

vaab commented Nov 20, 2011

What I missed was a way to get monospace without having to change code nor change URLs. I guess that a simple configure option server side would make me happy. And this would be an "Opt in" as suggested by @johnyma2. I don't see the point of having a button in the interface as it clutters it (and I would have it alway ON while others would have it always OFF).

@Wikinaut
Copy link
Contributor

Hello, the monospaced etherpad-lite pages are not rendered in monospace font when exported as PDF.
Can anyone fix this, too?

@JohnMcLear
Copy link
Member

@Wikinaut please file a separate bug report for that

@JohnMcLear JohnMcLear closed this Dec 4, 2011
@booo
Copy link
Contributor

booo commented Dec 22, 2011

This isn't part of the master branch? Could someone explain me why? I don't understand the decision making process...

For usability reason there should be a mono space button! Default for new pads should be mono spacing off.

@Wikinaut
Copy link
Contributor

@booo

my https://github.com/Wikinaut/etherpad-lite - which follows the master's core code changes - has "monospace" integrated.

@dohse
Copy link

dohse commented Dec 22, 2011

As we are running a etherpad instance for the use of mostly user with a computer science background. it would be really helpful for us to have at least the option to offer monospace font. it would a real hassle to maintain a patch for that.

i understand that you want to keep the ui clean and do not want to scare away users with to many knobs, but please offer the option for a monospace font button.

@jhollinger
Copy link
Contributor

Maybe a monospace button would make a good plugin, if those ever get off the ground.

@JohnMcLear
Copy link
Member

@booo Because the monospace button is used on < 5% of pads. Ideally it will be usable as a plugin like @jhollinger said

@booo
Copy link
Contributor

booo commented Jan 2, 2012

Can you proof the < 5%?

Several users/developers requested the feature or to be more precise a button to enable/disable the feature via the gui. Adding the button to the gui is a minimal change which doesn't heard anyone. I can't see the reason why we should not implement/add this button.

I think text layout and style is a core feature of the etherpad and shouldn't be part of any plugin.

@JohnMcLear
Copy link
Member

Yes

Can you proof the < 5%?

Several users/developers requested the feature or to be more precise a button to enable/disable the feature via the gui. Adding the button to the gui is a minimal change which doesn't heard anyone. I can't see the reason why we should not implement/add this button.

I think text layout and style is a core feature of the etherpad and shouldn't be part of any plugin.


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-3333012
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

@Wikinaut
Copy link
Contributor

Wikinaut commented Jan 2, 2012

We should perhaps make an opinion poll among the Etherpad lite developers, not the users (figures, no one can really "prove").

The addition of the monospace button is so easy, that I would rather suggest(!) to add the feature (like I did on my fork), and to have an settings.json for monospace = enabled than to have lengthy discussions.

What online tool is suited for opinion polls ?

@Wikinaut
Copy link
Contributor

Wikinaut commented Jan 2, 2012

Here's the edit button bar with "monospace" and "ordered list" buttons
https://github.com/Wikinaut/etherpad-lite/blob/master/static/img/etherpad_lite_icons.png

@jhollinger
Copy link
Contributor

While I am only partially against adding a monospace button to master, I do wonder how casually we should be adding config options to settings.json. An option to enable/disable an edit bar button (as I believe @Wikinaut is suggesting) strikes me as heading down a chaotic path. Is that just me? Don't mean to get off-topic, but it seems related.

@JohnMcLear
Copy link
Member

Ask developers not users.... bad idea IMHO

Wikinaut reply@reply.github.com wrote:

We should perhaps make an opinion poll among the Etherpad lite developers, not the users (figures, no one can really "prove").

The addition of the monospace button is so easy, that I would rather suggest(!) to add the feature (like I did on my fork), and to have an settings.json for monospace = enabled than to have lengthy discussions.

What online tool is suited for opinion polls ?


Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/pull/128#issuecomment-3333278
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

@Wikinaut
Copy link
Contributor

I suggest a behaviour change:

not changing a whole pad to "monospace" font what the current patch does,
but on span or line basis:

  1. mark a passage
  2. click "monospace" button
  3. only changing the marked passage to a fixed "monospace" font

@JohnMcLear
Copy link
Member

FYI This is in the latest version. git pull people.. git pull!

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.