Bug 388433

Summary:	Remove Warning Message When Changing Fonts
Product:	[Applications] systemsettings	Reporter:	Andres Betts <anditosan1000>
Component:	kcm_fonts	Assignee:	Unassigned bugs mailing-list <unassigned-bugs>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	bugseforuns, filipfila.kde, hein, kde, nate, unassigned-bugs
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://commits.kde.org/plasma-desktop/eae23d5d141b959a560b05a6a9403c1b0b188435	Version Fixed In:

Description Andres Betts 2018-01-01 19:27:29 UTC

Whenever the user changes fonts using system settings, the apply button will trigger a popup menu, where you have to click OK, to understand that effects will take only on new windows and may not be apparent immediately.

This can be removed.

Instead, it is best to leave a small message on the KCM itself that says something similar and avoid having the user click another OK button. It may give the impression that something went wrong.

Solution: Add message in KCM that says

"These changes will take effect on new windows."

Comment 1 Nate Graham 2018-01-02 16:53:27 UTC

Yeah,we should use an inline KMessageWidget for this instead of an intrusive pop-up window.

Comment 2 Kai Uwe Broulik 2018-01-08 09:47:57 UTC

Given this is being redone in QML, I wouldn't invest any effort in it right now but I agree. I did the same for appmenu settings where a message widget is used (obsolete now, but it was there :D)

Comment 3 Eike Hein 2018-04-04 09:21:10 UTC

*InlineMessage swoops in on a swinging rope*

Comment 4 Andres Betts 2018-04-04 14:03:10 UTC

+1

Comment 5 Filip Fila 2019-03-22 21:58:08 UTC

We can revisit this now, right?

Comment 6 Nate Graham 2019-03-24 15:53:12 UTC

Yep! It's QML now and using InlineMessage should be pretty simple.

Comment 7 Filip Fila 2019-07-26 14:55:59 UTC

Git commit eae23d5d141b959a560b05a6a9403c1b0b188435 by Filip Fila.
Committed on 26/07/2019 at 14:55.
Pushed by filipf into branch 'master'.

[kcms/fonts] Replace warning message with inline message when applying changes

Summary:
This patch gets rid of the popup message spawned when applying changes in the Fonts KCM and instead places the very same information in an inline message.

Test Plan: {F7097619}

Reviewers: #plasma, #vdg, ngraham, davidedmundson

Reviewed By: #plasma, #vdg, ngraham, davidedmundson

Subscribers: davidedmundson, broulik, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D22752

M  +3    -10   kcms/fonts/fonts.cpp
M  +1    -0    kcms/fonts/fonts.h
M  +194  -180  kcms/fonts/package/contents/ui/main.qml

https://commits.kde.org/plasma-desktop/eae23d5d141b959a560b05a6a9403c1b0b188435