> Everything looks good.
>
> Just one comment: an option in cssm with default value 0.6667 looks strange
> for me.
>
> I think it would be better if the value was between 0 and 255 (with default
> value = 170). Then, the code in UnityScreen::optionChanged would be something
> like:
>
> case UnityshellOptions::LauncherOpacity:
> launcher->SetBackgroundAlpha(optionGetLauncherOpacity() / 255.0);
> break;
I have your same opinion, but panel opacity has the same scale of values. So it may appear inconsistent.
> Everything looks good. :optionChanged would be something ns::LauncherOpa city: >SetBackgroundA lpha(optionGetL auncherOpacity( ) / 255.0);
>
> Just one comment: an option in cssm with default value 0.6667 looks strange
> for me.
>
> I think it would be better if the value was between 0 and 255 (with default
> value = 170). Then, the code in UnityScreen:
> like:
>
> case UnityshellOptio
> launcher-
> break;
I have your same opinion, but panel opacity has the same scale of values. So it may appear inconsistent.