dwp-forge / refnotes

4 stars 6 forks source link

Complete formating of inline notes in configuration #11

Closed dwp-forge closed 9 years ago

dwp-forge commented 9 years ago
It would be best to have complete formating of the inline notes in the
configuration editor. Currently superscript is not changeable.

Original issue reported on code.google.com by tammo.heeren on 2010-04-06 19:10:37

dwp-forge commented 9 years ago
Could you provide a use case?

Original issue reported on code.google.com by dwp-forge on 2010-04-06 23:02:31

dwp-forge commented 9 years ago
Issue 16 has been merged into this issue.

Original issue reported on code.google.com by dwp-forge on 2010-04-06 23:03:47

dwp-forge commented 9 years ago
Currently the [(:todo)] is _always_ superscript. I can not change it. The
configuration editor only allows me to change formating (italic,bold,etc). I'd like
to not have them superscript.

Original issue reported on code.google.com by tammo.heeren on 2010-04-09 21:46:36

dwp-forge commented 9 years ago
If it's only matter of taste, I would prefer to keep things as they are now. Removing
the superscript will be a breaking change, so don't want to go that way unless more
people will complain about it. And more importantly, I just like it that way ;)

If you really dislike superscript, you can remove <sup> tags in helper.php, line 456.

Original issue reported on code.google.com by dwp-forge on 2010-04-09 22:06:26

dwp-forge commented 9 years ago
Thank you for looking into it. The modification is easy enough to make on my end.  

Original issue reported on code.google.com by tammo.heeren on 2010-04-14 19:13:30

dwp-forge commented 9 years ago
I though it just required to move the <sub> definition into the configuration dialog
here italic, etc is also entered. I guess I could simply try that here by removing
the <sub> definition from the helper.php and to enter it in the configuration editor.

Original issue reported on code.google.com by tammo.heeren on 2010-04-14 19:15:52

dwp-forge commented 9 years ago
On the second thought, I decided to go ahead with your proposal. The next version is
going to be a major update, so I guess there won't be complains about breaking changes.

Implemented in r483.

Original issue reported on code.google.com by dwp-forge on 2012-05-13 11:11:03