jalr/OpenRGB
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Aligning Direction updates for consistency and readability

Browse source 
+ Adding check to see if the UI DirectionBox is enabled before executing changes to the box.
This commit is contained in:
Chris 2022-01-21 13:00:40 +11:00 • committed by Adam Honse
parent 5b4d55bb42
commit 6709cfb735
1 changed files with 33 additions and 30 deletions
Download patch file Download diff file Expand all files Collapse all files

63
qt/OpenRGBDevicePage.cpp
View file

@ -779,44 +779,47 @@ void Ui::OpenRGBDevicePage::UpdateMode()
bool supports_dir_hv = ( device->modes[(unsigned int)current_mode].flags & MODE_FLAG_HAS_DIRECTION_HV );
/*-----------------------------------------------------*\
| Set the direction value |
| If DirectionBox is enabled, set the direction values |
\*-----------------------------------------------------*/
if(supports_dir_hv)
if(ui->DirectionBox->isEnabled())
{
if(supports_dir_lr && supports_dir_ud)
if(supports_dir_hv)
{
if(supports_dir_lr && supports_dir_ud)
{
current_direction = current_dir_idx;
}
else if(supports_dir_lr || supports_dir_ud)
{
current_direction = current_dir_idx + 2;
}
else
{
current_direction = current_dir_idx + 4;
}
}
if(supports_dir_ud)
{
if(supports_dir_lr)
{
current_direction = current_dir_idx;
}
else
{
current_direction = current_dir_idx + 2;
}
}
if((supports_dir_lr)
&&(current_dir_idx < 2))
{
current_direction = current_dir_idx;
}
else if(supports_dir_lr || supports_dir_ud)
{
current_direction = current_dir_idx + 2;
}
else
{
current_direction = current_dir_idx + 4;
}
}
if(supports_dir_ud)
{
if(supports_dir_lr)
{
current_direction = current_dir_idx;
}
else
{
current_direction = current_dir_idx + 2;
}
device->modes[(unsigned int)current_mode].direction = current_direction;
}
if((supports_dir_lr)
&&(current_dir_idx < 2))
{
current_direction = current_dir_idx;
}
device->modes[(unsigned int)current_mode].direction = current_direction;
/*-----------------------------------------------------*\
| If Speed Slider is enabled, read the speed value |
\*-----------------------------------------------------*/