New plugin: ThinkPad - Fan & Thermal Control by Piero-93 · Pull Request #870 · noctalia-dev/noctalia-plugins

Piero-93 · 2026-05-18T12:37:16Z

Description

This PR adds the ThinkPad Fan Control (thinkpad-fan) plugin to the community registry.

It provides real-time hardware monitoring and control for Lenovo ThinkPad laptops by interfaces with /proc/acpi/ibm/fan and system thermal zones. The plugin has been fully updated to utilize Noctalia's native settings API and UX guidelines, integrating seamlessly with the system settings panel.

Features

Real-time Monitoring: Passive tracking of fan speed (RPM), current ACPI fan level, and CPU/System temperature.
Dynamic Coloring: Optional visual cues that change the bar capsule color based on fan status (e.g., when the fan is stopped or set to a custom override level).
Native Integration: matching Noctalia's UI design language
Compatible with every laptop with /proc/acpi/ibm/fan interface.

Plugin Structure

thinkpad-fan/
├── manifest.json      # Metadata with default settings declared for persistence
├── preview.png        # 16:9 preview image
├── README.md          # Documentation and configuration requirements
├── BarWidget.qml      # Main bar UI component
└── Settings.qml       # Native settings module

Test

Tested on 2 ThinkPad T14 Gen 1.

- Percentage & remaining time - Popup controls in a padded pill

…override the fan mode

spiros132 · 2026-05-21T22:02:56Z

This PR also contains your other PR about the battery-and-power-management plugin. Prefer to have one PR for one plugin.

Piero-93 · 2026-05-22T09:56:18Z

Hi, thanks, I'm sorry but this are my very first PRs and I'm not very skilled with git. I'll work on it!

spiros132 · 2026-05-24T13:29:36Z

Hi, thanks, I'm sorry but this are my very first PRs and I'm not very skilled with git. I'll work on it!

No worries at all, if you need any help just ask! :)

spiros132

Some feedback about the PR! :D

spiros132 · 2026-05-24T13:30:32Z

+    readonly property string screenName: screen ? (screen.name ?? "") : ""
+    readonly property real capsuleHeight: (typeof Style !== "undefined" && typeof Style.getCapsuleHeightForScreen === "function") ? Style.getCapsuleHeightForScreen(root.screenName) : 26
+    readonly property real barFontSize: (typeof Style !== "undefined" && typeof Style.getBarFontSizeForScreen === "function") ? Style.getBarFontSizeForScreen(root.screenName) : 10
+    readonly property string fixedFont: (typeof Settings !== "undefined" && Settings.data?.ui?.fontFixed) ? Settings.data.ui.fontFixed : "monospace"


Settings and Style should never be undefined. If they are then something has gone wrong. You don't need to check if they are undefined or not.

spiros132 · 2026-05-24T13:30:55Z

+        pluginApi?.manifest?.metadata?.defaultSettings?.allowPopupOpening ??
+        true
+
+    readonly property real contentWidth: layout.implicitWidth + ((typeof Style !== "undefined") ? Style.marginS * 2 : 8)


Here as well, you don't need to check if Style is undefined or not.

spiros132 · 2026-05-24T13:31:02Z

+    readonly property real contentWidth: layout.implicitWidth + ((typeof Style !== "undefined") ? Style.marginS * 2 : 8)
+    readonly property real contentHeight: capsuleHeight
+    implicitWidth: contentWidth
+    implicitHeight: (typeof Style !== "undefined") ? Style.barHeight : 32


Here either

spiros132 · 2026-05-24T13:31:51Z

+import QtQuick.Layouts
+import QtQuick.Controls
+import Quickshell
+import Quickshell.Io as QSIo


Is there a reason why this needs to be imported as a separate name?

spiros132 · 2026-05-24T13:32:26Z

+    }
+
+    // Hardware fan monitoring (passive tracking bound to thinkfan)
+    QSIo.FileView {


You can just use the regular FileView no need to specify where the component comes from.

spiros132 · 2026-05-24T13:37:04Z

+    }
+
+    Timer { id: refreshTimer; interval: 300; repeat: false; onTriggered: { fanLoader.reload(); tempLoader.reload(); } }
+    Timer { interval: 2000; running: true; repeat: true; triggeredOnStart: true; onTriggered: { fanLoader.reload(); tempLoader.reload(); } }


Is this timer really needed? I believe that the FileView component gets updated immediately when any change happens on the file.

spiros132 · 2026-05-24T13:38:02Z

+        anchors.centerIn: parent
+        width: root.contentWidth
+        height: root.contentHeight
+        radius: (typeof Style !== "undefined") ? Style.radiusL : 6


As before, you don't need to check if Style is undefined or not.

spiros132 · 2026-05-24T13:38:52Z

+            : (root.isCustomActive
+                ? ((typeof Color !== "undefined") ? Color.mPrimary : "#3355ff")
+                : (root.fanLevel === "0" ? "#cc241d" : ((typeof Style !== "undefined") ? Style.capsuleBorderColor : "#33ffffff")))
+        border.width: (typeof Style !== "undefined") ? Style.capsuleBorderWidth : 1


You don't need to check if Style or any other singleton component exists on any of these.

I won't comment on each and every one to skip spam.

spiros132 · 2026-05-24T13:40:22Z

+            pointSize: (typeof Style !== "undefined") ? Style.fontSizeS : 10
+            color: parent.isSelected
+                ? ((typeof Color !== "undefined") ? Color.mOnPrimary : "#ffffff")
+                : ((typeof Color !== "undefined") ? Color.mOnSurface : "#ffffff")


As before, no need to check if these singleton are undefined or not.

spiros132 · 2026-05-24T13:41:29Z

+
+Grant group write parameters over systemic thermal interfaces by applying the setup script:
+
+    cd ~/.config/noctalia/plugins/fan-speed/


This isn't the correct plugin name.

Piero-93 added 12 commits May 15, 2026 22:05

V1: perc + watts + tooltip

50aa5f9

Better graphics, restored tooltip, first test on popup

1b94589

Removed duplicated files

1dfc4fa

Better popup appearance

640de8c

- Percentage & remaining time - Popup controls in a padded pill

Battery threshold & style fixes

f29f475

Renamed and ready for PR

eb48c0c

Renamed in manifest

a5f28c9

Shows fan speed (with color code for different statuses) and let you …

bb3a08f

…override the fan mode

Settings window (settings are non-persistent yet)

4203baa

Settings are persistent

9a38ba4

Fix + preview image

f76e46c

Manifest compliancy to guidelines

c9e0079

This comment was marked as resolved.

Sign in to view

spiros132 marked this pull request as draft May 18, 2026 17:59

Fix after PR

f9082be

Piero-93 marked this pull request as ready for review May 18, 2026 20:07

Removed another new widget pushed by error

a83ed64

This comment was marked as resolved.

Sign in to view

Piero-93 marked this pull request as draft May 22, 2026 09:56

Fixed some code style problems in FanPopup and SettingsView

fde5780

Piero-93 marked this pull request as ready for review May 22, 2026 10:06

spiros132 reviewed May 24, 2026

View reviewed changes

spiros132 marked this pull request as draft May 24, 2026 13:43


		Grant group write parameters over systemic thermal interfaces by applying the setup script:

		cd ~/.config/noctalia/plugins/fan-speed/

Conversation

Piero-93 commented May 18, 2026

Description

Features

Plugin Structure

Test

Uh oh!

This comment was marked as resolved.

spiros132 commented May 21, 2026

Uh oh!

This comment was marked as resolved.

Piero-93 commented May 22, 2026

Uh oh!

spiros132 commented May 24, 2026

Uh oh!

spiros132 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants