1
0
mirror of https://github.com/qTox/qTox.git synced 2024-03-22 14:00:36 +08:00
Commit Graph

8 Commits

Author SHA1 Message Date
sudden6
e9515f4898
fix: remove useless line that caused a warning 2017-03-23 13:36:29 +01:00
noavarice
08208e9aa5 fix: fixed wrong formatting for multiple URL's in one message
Fix #4275
I did not consider that replacing one substring with another will point
to shifting position of next URL found with regexp. That's the behavior
of Qt's "QRegularExpression" class - it takes a string into constructor
and seems to make its copy inside so changing source string does not
affect this regex object
2017-03-22 14:23:57 +03:00
noavarice
18c3f3a217 fix: URL's are not affected by markdown
Fix #4242
Just running in a loop through all of markdown symbols in found URL's,
replacing them with their associated HTML character code before the code
formatting to avoid any chance of interception
2017-03-12 23:14:44 +03:00
noavarice
e8ed7e1f53 fix: fixed documentation mistake 2017-03-07 15:34:52 +03:00
noavarice
3e2bfdd548 refactor: further improvement of message formatting
Brief list of changes:
-  names of some variables and constants were replaced for reading
   convenience;
-  URL-highlighting method moved to TextFormatter so URL's don't
   conflict with italic text formatting;
-  as I understand, in previous version 'file://' URL does not work
   because of bad regex. I fix this with help of wiki page that
   referenced in comment for old code. Important note: there are two
   equal 'file://' URL syntax: 'file:///...' and 'file://localhost/...'
   and the second one does NOT work in KDE but works in Gnome so that's
   not a bug
Fix #4233
2017-03-06 11:55:54 +03:00
Zetok Zalbavar
80f5de31b3
style: reformat current C++ codebase using clang-format 2017-02-26 11:52:45 +00:00
noavarice
87f219a78f refactor: message text formatting works better now
- tag intersection detected
- variables and constants' names became shorter
2017-02-19 23:53:11 +03:00
noavarice
5047a65e11 refactor: improved HTML-formatting mechanism for text messages
Added class provides text font formatting with HTML font tags
Supported stuff:
  - nested formatting;
  - several text pieces formatted with the same formatting style at the
    one message;
  - styling applies only if non-whitespace symbol follows opening
    formatting symbol and also non-whitespace symbol preceds closing
    symbol;
  - only multiline code font formatting supports new line inside of
    message text.
Also fix #3804
2017-02-19 10:06:26 +03:00