broulik added a comment.

  +1 to the idea

INLINE COMMENTS

> SettingsBase.cpp:151
> +    systemInformationAction = actionCollection()->addAction( 
> QStringLiteral("help_about_system"), this, [] () {
> +        KRun::runCommand( QStringLiteral("kinfocenter"), nullptr );
> +    } );

Can you do `runApplication` with `org.kde.kinfocenter` service, then you can 
also just read the icon etc from there

Also, perhaps check if it is installed?

> SettingsBase.cpp:402
>          QList<QAction*> configureBarActions;
> -        configureBarActions << configureAction;
> +        configureBarActions << configureAction << systemInformationAction;
>          guiFactory()->plugActionList( this, QStringLiteral("configure"), 
> configureBarActions );

Not sure about having a bigass button with wide text in the toolbar at all times

REPOSITORY
  R124 System Settings

REVISION DETAIL
  https://phabricator.kde.org/D22728

To: ngraham, #plasma, #vdg
Cc: broulik, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, 
Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, mart

Reply via email to