mirror of
https://gitlab.com/asus-linux/asusctl.git
synced 2026-02-06 00:15:04 +01:00
feat(rog-control-center): Major UI/UX improvements and new features
- Add software RGB animations for static-only keyboards (rainbow, color cycle) - Add custom fan curve control via direct sysfs for unsupported laptops - Add real-time system status bar (CPU/GPU temps, fan speeds, power draw) - Add tray icon tooltip with live system stats - Add power profile change notifications (Fn+F5) - Add dGPU status notifications - Add ROG theme with dark palette and accent colors - Add Screenpad, Slash, and SuperGFX page stubs - Improve fan curve graph UI - Various UI refinements and fixes Co-Authored-By: Gemini <noreply@google.com>
This commit is contained in:
@@ -1,4 +1,6 @@
|
||||
use std::sync::{Arc, Mutex};
|
||||
use crate::ui::show_toast;
|
||||
use std::collections::HashMap;
|
||||
use std::sync::{Arc, Mutex, OnceLock};
|
||||
|
||||
use log::error;
|
||||
use rog_dbus::zbus_fan_curves::FanCurvesProxy;
|
||||
@@ -8,7 +10,25 @@ use rog_profiles::fan_curve_set::CurveData;
|
||||
use slint::{ComponentHandle, Model, Weak};
|
||||
|
||||
use crate::config::Config;
|
||||
use crate::{FanPageData, FanType, MainWindow, Node};
|
||||
use crate::{FanPageData, FanType, MainWindow, Node, Profile};
|
||||
|
||||
// Isolated Rust-side cache for fan curves (not affected by Slint reactivity)
|
||||
type FanCacheKey = (i32, i32); // (Profile as i32, FanType as i32)
|
||||
static FAN_CACHE: OnceLock<Mutex<HashMap<FanCacheKey, Vec<Node>>>> = OnceLock::new();
|
||||
|
||||
fn fan_cache() -> &'static Mutex<HashMap<FanCacheKey, Vec<Node>>> {
|
||||
FAN_CACHE.get_or_init(|| Mutex::new(HashMap::new()))
|
||||
}
|
||||
|
||||
fn cache_fan_curve(profile: Profile, fan_type: FanType, nodes: Vec<Node>) {
|
||||
let key = (profile as i32, fan_type as i32);
|
||||
fan_cache().lock().unwrap().insert(key, nodes);
|
||||
}
|
||||
|
||||
fn get_cached_fan_curve(profile: Profile, fan_type: FanType) -> Option<Vec<Node>> {
|
||||
let key = (profile as i32, fan_type as i32);
|
||||
fan_cache().lock().unwrap().get(&key).cloned()
|
||||
}
|
||||
|
||||
pub fn update_fan_data(
|
||||
handle: Weak<MainWindow>,
|
||||
@@ -19,7 +39,7 @@ pub fn update_fan_data(
|
||||
handle
|
||||
.upgrade_in_event_loop(move |handle| {
|
||||
let global = handle.global::<FanPageData>();
|
||||
let collect = |temp: &[u8], pwm: &[u8]| -> slint::ModelRc<Node> {
|
||||
let _collect = |temp: &[u8], pwm: &[u8]| -> slint::ModelRc<Node> {
|
||||
let tmp: Vec<Node> = temp
|
||||
.iter()
|
||||
.zip(pwm.iter())
|
||||
@@ -33,61 +53,100 @@ pub fn update_fan_data(
|
||||
|
||||
for fan in bal {
|
||||
global.set_balanced_available(true);
|
||||
let nodes_vec: Vec<Node> = fan
|
||||
.temp
|
||||
.iter()
|
||||
.zip(fan.pwm.iter())
|
||||
.map(|(x, y)| Node {
|
||||
x: *x as f32,
|
||||
y: *y as f32,
|
||||
})
|
||||
.collect();
|
||||
let nodes: slint::ModelRc<Node> = nodes_vec.as_slice().into();
|
||||
match fan.fan {
|
||||
rog_profiles::FanCurvePU::CPU => {
|
||||
global.set_cpu_fan_available(true);
|
||||
global.set_balanced_cpu_enabled(fan.enabled);
|
||||
global.set_balanced_cpu(collect(&fan.temp, &fan.pwm))
|
||||
global.set_balanced_cpu(nodes.clone());
|
||||
cache_fan_curve(Profile::Balanced, FanType::CPU, nodes_vec);
|
||||
}
|
||||
rog_profiles::FanCurvePU::GPU => {
|
||||
global.set_gpu_fan_available(true);
|
||||
global.set_balanced_gpu_enabled(fan.enabled);
|
||||
global.set_balanced_gpu(collect(&fan.temp, &fan.pwm))
|
||||
global.set_balanced_gpu(nodes.clone());
|
||||
cache_fan_curve(Profile::Balanced, FanType::GPU, nodes_vec);
|
||||
}
|
||||
rog_profiles::FanCurvePU::MID => {
|
||||
global.set_mid_fan_available(true);
|
||||
global.set_balanced_mid_enabled(fan.enabled);
|
||||
global.set_balanced_mid(collect(&fan.temp, &fan.pwm))
|
||||
global.set_balanced_mid(nodes.clone());
|
||||
cache_fan_curve(Profile::Balanced, FanType::Middle, nodes_vec);
|
||||
}
|
||||
}
|
||||
}
|
||||
for fan in perf {
|
||||
global.set_performance_available(true);
|
||||
let nodes_vec: Vec<Node> = fan
|
||||
.temp
|
||||
.iter()
|
||||
.zip(fan.pwm.iter())
|
||||
.map(|(x, y)| Node {
|
||||
x: *x as f32,
|
||||
y: *y as f32,
|
||||
})
|
||||
.collect();
|
||||
let nodes: slint::ModelRc<Node> = nodes_vec.as_slice().into();
|
||||
match fan.fan {
|
||||
rog_profiles::FanCurvePU::CPU => {
|
||||
global.set_cpu_fan_available(true);
|
||||
global.set_performance_cpu_enabled(fan.enabled);
|
||||
global.set_performance_cpu(collect(&fan.temp, &fan.pwm))
|
||||
global.set_performance_cpu(nodes.clone());
|
||||
cache_fan_curve(Profile::Performance, FanType::CPU, nodes_vec);
|
||||
}
|
||||
rog_profiles::FanCurvePU::GPU => {
|
||||
global.set_gpu_fan_available(true);
|
||||
global.set_performance_gpu_enabled(fan.enabled);
|
||||
global.set_performance_gpu(collect(&fan.temp, &fan.pwm))
|
||||
global.set_performance_gpu(nodes.clone());
|
||||
cache_fan_curve(Profile::Performance, FanType::GPU, nodes_vec);
|
||||
}
|
||||
rog_profiles::FanCurvePU::MID => {
|
||||
global.set_mid_fan_available(true);
|
||||
global.set_performance_mid_enabled(fan.enabled);
|
||||
global.set_performance_mid(collect(&fan.temp, &fan.pwm))
|
||||
global.set_performance_mid(nodes.clone());
|
||||
cache_fan_curve(Profile::Performance, FanType::Middle, nodes_vec);
|
||||
}
|
||||
}
|
||||
}
|
||||
for fan in quiet {
|
||||
global.set_quiet_available(true);
|
||||
let nodes_vec: Vec<Node> = fan
|
||||
.temp
|
||||
.iter()
|
||||
.zip(fan.pwm.iter())
|
||||
.map(|(x, y)| Node {
|
||||
x: *x as f32,
|
||||
y: *y as f32,
|
||||
})
|
||||
.collect();
|
||||
let nodes: slint::ModelRc<Node> = nodes_vec.as_slice().into();
|
||||
match fan.fan {
|
||||
rog_profiles::FanCurvePU::CPU => {
|
||||
global.set_cpu_fan_available(true);
|
||||
global.set_quiet_cpu_enabled(fan.enabled);
|
||||
global.set_quiet_cpu(collect(&fan.temp, &fan.pwm))
|
||||
global.set_quiet_cpu(nodes.clone());
|
||||
cache_fan_curve(Profile::Quiet, FanType::CPU, nodes_vec);
|
||||
}
|
||||
rog_profiles::FanCurvePU::GPU => {
|
||||
global.set_gpu_fan_available(true);
|
||||
global.set_quiet_gpu_enabled(fan.enabled);
|
||||
global.set_quiet_gpu(collect(&fan.temp, &fan.pwm))
|
||||
global.set_quiet_gpu(nodes.clone());
|
||||
cache_fan_curve(Profile::Quiet, FanType::GPU, nodes_vec);
|
||||
}
|
||||
rog_profiles::FanCurvePU::MID => {
|
||||
global.set_mid_fan_available(true);
|
||||
global.set_quiet_mid_enabled(fan.enabled);
|
||||
global.set_quiet_mid(collect(&fan.temp, &fan.pwm))
|
||||
global.set_quiet_mid(nodes.clone());
|
||||
cache_fan_curve(Profile::Quiet, FanType::Middle, nodes_vec);
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -171,6 +230,7 @@ pub fn setup_fan_curve_page(ui: &MainWindow, _config: Arc<Mutex<Config>>) {
|
||||
let choices_for_ui = platform_profile_choices.clone();
|
||||
let handle_next1 = handle_copy.clone();
|
||||
if let Err(e) = handle_copy.upgrade_in_event_loop(move |handle| {
|
||||
let handle_weak_for_fans = handle.as_weak();
|
||||
let global = handle.global::<FanPageData>();
|
||||
let fans1 = fans.clone();
|
||||
let choices = choices_for_ui.clone();
|
||||
@@ -212,17 +272,103 @@ pub fn setup_fan_curve_page(ui: &MainWindow, _config: Arc<Mutex<Config>>) {
|
||||
update_fan_data(handle_next, balanced, perf, quiet);
|
||||
});
|
||||
});
|
||||
|
||||
let handle_weak_for_cancel = handle_weak_for_fans.clone();
|
||||
global.on_set_fan_data(move |fan, profile, enabled, data| {
|
||||
if crate::ui::setup_fan_curve_custom::is_custom_fan_supported() {
|
||||
let handle_weak = handle_weak_for_fans.clone();
|
||||
let data: Vec<Node> = data.iter().collect();
|
||||
|
||||
use log::info;
|
||||
info!("MainThread: Request to apply custom curve for {:?}", fan);
|
||||
|
||||
// Explicitly spawn a thread to handle this, preventing ANY main thread blocking
|
||||
std::thread::spawn(move || {
|
||||
info!("WorkerThread: applying curve for {:?}", fan);
|
||||
crate::ui::setup_fan_curve_custom::apply_custom_fan_curve(
|
||||
handle_weak.clone(),
|
||||
fan,
|
||||
enabled,
|
||||
data,
|
||||
);
|
||||
info!("WorkerThread: returned from apply (async), clearing busy flag for {:?}", fan);
|
||||
|
||||
// Clear busy flag
|
||||
let _ = handle_weak.upgrade_in_event_loop(move |h| {
|
||||
let g = h.global::<FanPageData>();
|
||||
match fan {
|
||||
FanType::CPU => g.set_is_busy_cpu(false),
|
||||
FanType::GPU => g.set_is_busy_gpu(false),
|
||||
FanType::Middle => g.set_is_busy_mid(false),
|
||||
}
|
||||
info!("MainThread: cleared busy flag for {:?}", fan);
|
||||
});
|
||||
});
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
let fans = fans.clone();
|
||||
let data: Vec<Node> = data.iter().collect();
|
||||
let data = fan_data_for(fan, enabled, data);
|
||||
let handle_weak = handle_weak_for_fans.clone();
|
||||
let nodes_vec: Vec<Node> = data.iter().collect();
|
||||
let _data_copy = nodes_vec.clone();
|
||||
let cache_copy = nodes_vec.clone(); // Clone for cache update
|
||||
let fan_data = fan_data_for(fan, enabled, nodes_vec);
|
||||
tokio::spawn(async move {
|
||||
fans.set_fan_curve(profile.into(), data)
|
||||
.await
|
||||
.map_err(|e| error!("{e:}"))
|
||||
.ok()
|
||||
show_toast(
|
||||
"Fan curve applied".into(),
|
||||
"Failed to apply fan curve".into(),
|
||||
handle_weak.clone(),
|
||||
fans.set_fan_curve(profile.into(), fan_data).await,
|
||||
);
|
||||
let _ = handle_weak.upgrade_in_event_loop(move |h| {
|
||||
let g = h.global::<FanPageData>();
|
||||
// Update Rust-side cache (isolated from Slint properties)
|
||||
cache_fan_curve(profile, fan, cache_copy);
|
||||
|
||||
match fan {
|
||||
FanType::CPU => g.set_is_busy_cpu(false),
|
||||
FanType::GPU => g.set_is_busy_gpu(false),
|
||||
FanType::Middle => g.set_is_busy_mid(false),
|
||||
}
|
||||
});
|
||||
});
|
||||
});
|
||||
global.on_cancel(move |fan_type, profile| {
|
||||
let handle_weak = handle_weak_for_cancel.clone();
|
||||
let _ = handle_weak.upgrade_in_event_loop(move |h: MainWindow| {
|
||||
let global = h.global::<FanPageData>();
|
||||
|
||||
// Retrieve from isolated Rust cache
|
||||
let nodes_opt = get_cached_fan_curve(profile, fan_type);
|
||||
|
||||
if let Some(nodes_vec) = nodes_opt {
|
||||
use log::info;
|
||||
info!("Canceling {:?} {:?} - restoring {} nodes from isolated cache", fan_type, profile, nodes_vec.len());
|
||||
|
||||
let new_model: slint::ModelRc<Node> = nodes_vec.as_slice().into();
|
||||
|
||||
match (profile, fan_type) {
|
||||
(crate::Profile::Balanced, FanType::CPU) => global.set_balanced_cpu(new_model),
|
||||
(crate::Profile::Balanced, FanType::GPU) => global.set_balanced_gpu(new_model),
|
||||
(crate::Profile::Balanced, FanType::Middle) => global.set_balanced_mid(new_model),
|
||||
(crate::Profile::Performance, FanType::CPU) => global.set_performance_cpu(new_model),
|
||||
(crate::Profile::Performance, FanType::GPU) => global.set_performance_gpu(new_model),
|
||||
(crate::Profile::Performance, FanType::Middle) => global.set_performance_mid(new_model),
|
||||
(crate::Profile::Quiet, FanType::CPU) => global.set_quiet_cpu(new_model),
|
||||
(crate::Profile::Quiet, FanType::GPU) => global.set_quiet_gpu(new_model),
|
||||
(crate::Profile::Quiet, FanType::Middle) => global.set_quiet_mid(new_model),
|
||||
_ => {}
|
||||
}
|
||||
} else {
|
||||
log::warn!("Cancel failed: No cached data for {:?} {:?}", fan_type, profile);
|
||||
}
|
||||
});
|
||||
});
|
||||
// Initialize warning
|
||||
if crate::ui::setup_fan_curve_custom::is_custom_fan_supported() {
|
||||
global.set_show_custom_warning(true);
|
||||
}
|
||||
}) {
|
||||
error!("setup_fan_curve_page: upgrade_in_event_loop: {e:?}");
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user