Small change: using jquerys .html() instead of .text() to enable usage of html entities#43
Small change: using jquerys .html() instead of .text() to enable usage of html entities#43haschek wants to merge 1 commit intormm5t:masterfrom haschek:patch-1
Conversation
…acter ( ), this is important to prevent a line break between the number and the timeframe
|
Thanks for taking the time to submit this, but I'm sorry, I'm not going to pull this in. Reasons:
It would be better to use CSS to manage whether you want the timestamps to wrap or not. For example, you can prevent word wrapping with something like this: or if you are willing to use a deprecated HTML tag, you can wrap the timestamp between |
|
Ok, thx for taking the time to check the pull request and answering. Let me explain it more detailed: using the in the text settings would be optional, but if someone want to use it, .text() can't handle this, using .html() is necessary.
Right, maybe. Even this is a typographical no-go :) Number and unit (timeframe) should not be splitted up on two lines.
No, that's not mandatory. Using the non-breaking-space char is only a option.
Maybe, but's the HTML entity for the non-breaking-space character what is a typographical sign on its own.
That would be possible but it's not the same. If you are using regards, |
|
Haschek, Okay, you've made a strong enough argument for me to reopen this. Let me give it some thought to make sure there wasn't a good reason for me using |
|
Maybe this helps? http://stackoverflow.com/questions/1910794/jquery-text-vs-html/ Or this? |
|
@rmm5t Any updates on this PR? |
[mod] improving typography: enabling usage of non-breaking-space character ( ) in textfields, this is important to prevent a line break between the number and the timeframe