Hello,

First of all i want to say, that i am new to the LXQt project  to
opensource development in general.
But i want to learn something and my laptops brightness keys don't work
with lxqt.
Looking at https://github.com/lxde/lxqt/issues/36 i wondered if
backlight control should be integrated into the lxqt-powermanagement
project or, similar to volume control, as a plugin.

While browsing the lxqt-panel and plugin code i noticed that there are
currently two ways to get the paths to the directory where the plugins
should be found:

In plugin.cpp line 75-77:
QStringList dirs;
dirs <<
QProcessEnvironment::systemEnvironment().value("LXQTPANEL_PLUGIN_PATH").split(":");
dirs << PLUGIN_DIR;

And in lxqtpanel.cpp line 295-297
dirs << QString(getenv("LXQT_PANEL_PLUGINS_DIR")).split(':',
QString::SkipEmptyParts);
dirs << QString("%1/%2").arg(XdgDirs::dataHome(), "/lxqt/lxqt-panel");
dirs << PLUGIN_DESKTOPS_DIR;

So when i tried to get the code to run with the volume control plugin,
lxqtpanel found it (because the LXQT_PANEL_PLUGINS_DIR was set) but
plugin.cpp didn't find it.
Is that a bug or is there a reason for that?


Additionaly to that i have two more general questions:
1. Is this the right place to post these kinds of questions?
2. At the moment i think the backlight control code should be inside the
powermanagement project.
Reasons: Backlight control should work without plugin and the code
belongs to that area semantically, imo.
Would you agree?

Best regards,

Jakob

------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk
_______________________________________________
Lxde-list mailing list
Lxde-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxde-list

Reply via email to