Some apps can't handle the RLM/LRM characters used in the textview when copied.
This shall decrease the use to it's minimum.
Details
Details
- Reviewers
argonel - Group Reviewers
Konversation - Commits
- R7:8384d546baa8: Minimze the use of bidi markers in IRCView.
Needs more testing.
Diff Detail
Diff Detail
- Repository
- R7 Konversation
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage
Comment Actions
The blocks starting at 565, 623 and 683 and possibly 801 look like duplicates to me. Perhaps factor these out into a helper method? I didn't actually compare every line, so if they're not strict duplicates ignore me.
src/viewer/ircview.cpp | ||
---|---|---|
605 | The comments like this probably mean something, please expand into something more like documentation. |
Comment Actions
Ok, I applied the comment, but now I have no idea how to merge, rebase and other stuff, sorry! :(
Please do it instead since my knowledge in Git is below zero.
Thanks for the hard work! :-)