D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Kai Uwe Broulik
broulik added a comment. Also keep in mind that Qt 5.14 supposedly supports markdown editing, which I would love in the notes applet, which would be preferred over "rich text" imho. REPOSITORY R114 Plasma Addons BRANCH notes-applet-tuning (branched from master) REVISION DETAIL

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Vladimir Yerilov
yerilov added a comment. @ngraham yeah this is the best way to go as it seems to me. What I did was just drawing attention to this problem since I cannot code and it's hard to understand how everything works for a non-coder. @ltoscano I'd vote for any solution which makes Notes more

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Nathaniel Graham
ngraham added a comment. In D24181#538206 , @ltoscano wrote: > @ngraham Let's make it configurable, but not change the default then. I disagree that changing this default on an application which support rich text by default helps making the

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Luigi Toscano
ltoscano added a comment. @ngraham Let's make it configurable, but not change the default then. I disagree that changing this default on an application which support rich text by default helps making the life of everyone better. @yerilov, as you can see, I didn't request a change,

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Vladimir Yerilov
yerilov added a comment. @ltoscano D22774 added the ability to choose between paste with and without formatting in the context menu, so yes, no problem with distinguishing. Adding another shortcut like Ctrl+Shift+V is also a viable solution but these

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Nathaniel Graham
ngraham added a comment. In D24181#538185 , @ltoscano wrote: > Please ignore the last part of my last comment. My question on the way a paste happens is still relevant, so please answer it: if users use the right click, they will surely

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Luigi Toscano
ltoscano added a comment. Please ignore the last part of my last comment. My question on the way a paste happens is still relevant, so please answer it: if users use the right click, they will surely distiguish "paste" and "paste without formatting". If they use the shortcut, they can learn

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Luigi Toscano
ltoscano added a comment. Please also note that the notes applications is a rich text editor by default (like the buttons shows). So I would vote for a revert of D22774 . REPOSITORY R114 Plasma Addons BRANCH notes-applet-tuning (branched from master)

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Luigi Toscano
ltoscano added a comment. I don't see how using the "running in circles" argument can solve the problem. I totally understand how notes works, thanks. The situation is how I described above: applications with just paste may have non-formatting option; applications which have both options

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Vladimir Yerilov
yerilov added a comment. > Do those applications provide the paste-with-formatting action too? No they don't. So it is one more thing that actually makes Plasma more feature-rich DE. > When you talk about "pastes text", "paste it", etc, which way do you mean? Well, we're

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Luigi Toscano
ltoscano added a comment. Do those applications provide the paste-with-formatting action too? When you talk about "pastes text", "paste it", etc, which way do you mean? Shortcut, right click + "Paste" entry, middle menu click, all of them, some of them? REPOSITORY R114 Plasma Addons

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-26 Thread Vladimir Yerilov
yerilov added a comment. Talking about cross-platform: Windows Notes app pastes text with no formatting, as Gnome Sticky Notes do, even Android apps do not use pasting with formatting by default. Yep, I understand expecting pasting with formatting for Office apps, but for Notes widget it

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-25 Thread Luigi Toscano
ltoscano added a comment. I think that the Kate use case is different, as it provides only one paste action. Whenever else there are two paste actions available, it's always "paste" (with formatting) associated to ctrl-V and "paste without formatting" to ctrl-shift-V. So maybe you are

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-25 Thread Vladimir Yerilov
yerilov added a comment. Funny thing: I stopped using Notes due to this formatting thing so long ago that I cannot remember when exactly. There's a plaintext Kate file on my desktop instead. REPOSITORY R114 Plasma Addons BRANCH notes-applet-tuning (branched from master) REVISION

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-25 Thread Vladimir Yerilov
yerilov added a comment. @huftis Having thought about this I guess I have my attitude towards the default `Ctlr+V` 'Paste' action: it should be without formatting. I can explain why. Notes applet is not an application. The reason it exists is to contain quick notes and tasks, memos, etc.

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-25 Thread Nathaniel Graham
ngraham added a comment. In D24181#537896 , @davidedmundson wrote: > > If this behavior is considered objectionable and people hate it (personally I like it), perhaps an alternative approach would be to add a UI to toggle individual notes

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-25 Thread David Edmundson
davidedmundson added a comment. > If this behavior is considered objectionable and people hate it (personally I like it), perhaps an alternative approach would be to add a UI to toggle individual notes between rich text and plain text, with plain text being the default? See

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-25 Thread Nathaniel Graham
ngraham added a comment. The reason we decided to make this change was because users were complaining that pasting text into the Notes widget preserved formatting. See https://bugs.kde.org/show_bug.cgi?id=396507. So clearly not everyone expected that. It's the classic conflict between

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-25 Thread Karl Ove Hufthammer
huftis added a comment. I find ‘Ctrl + V’ doing a ‘paste without formatting’ (with or without the renaming) confusing, and it goes against the principle of least surprise, which is in general a bad idea. In all(?) other applications, ‘Ctrl + V’ does a normal paste, i.e. with formatting (if

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-25 Thread Luigi Toscano
ltoscano added a comment. I think that this requires a bit more of discussion; "Paste" has a specific and well expected meaning. REPOSITORY R114 Plasma Addons BRANCH notes-applet-tuning (branched from master) REVISION DETAIL https://phabricator.kde.org/D24181 To: yerilov, ngraham

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-24 Thread Nathaniel Graham
ngraham added a comment. In D24181#536786 , @broulik wrote: > In D24181#536785 , @ltoscano wrote: > > > Out of curiosity: doesn't usually the default Paste action (ctrl-V) paste with formatting?

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-24 Thread Luigi Toscano
ltoscano added a comment. What changes is the patch needed. If we agree that Paste means "paste with formatting", then the string don't need to be changed, just the shortcut. REPOSITORY R114 Plasma Addons BRANCH notes-applet-tuning (branched from master) REVISION DETAIL

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-24 Thread Vladimir Yerilov
yerilov added a comment. I have no objections regarding default action, be it paste with or w/o formatting, it's rather a common sense fix: name what we use for Ctrl+V as "Paste", but and another action should get more detailed description. It also looks good in menu:

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-24 Thread Kai Uwe Broulik
broulik added a comment. In D24181#536785 , @ltoscano wrote: > Out of curiosity: doesn't usually the default Paste action (ctrl-V) paste with formatting? Shouldn't the shortcut be changed instead? Yeah, normally Ctrl+Shift+V pastes

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-23 Thread Luigi Toscano
ltoscano added a comment. Out of curiosity: doesn't usually the default Paste action (ctrl-V) paste with formatting? Shouldn't the shortcut be changed instead? REPOSITORY R114 Plasma Addons BRANCH notes-applet-tuning (branched from master) REVISION DETAIL

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-23 Thread Vladimir Yerilov
yerilov added a comment. Thanks Nate :) Actually it's rational to change `documentHandler.pasteWithoutFormatting()` handler as well, but I'm afraid I'm not good at coding to make such amendments. REPOSITORY R114 Plasma Addons BRANCH notes-applet-tuning (branched from master) REVISION

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-23 Thread Nathaniel Graham
ngraham added subscribers: Localization, aacid, ltoscano, yurchor. ngraham added a comment. Thanks for the patch! May it be the first of many. :) #localization team, I'd like to request a string freeze exception and land this on the

D24181: Changing "Paste Without Formatting" in Notes applet to "Paste"

2019-09-23 Thread Vladimir Yerilov
yerilov created this revision. yerilov added a reviewer: ngraham. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. yerilov requested review of this revision. REVISION SUMMARY Since this action is the default for Ctrl+V shortcut it should be named just "Paste". Hence the