Address review feedback (#4485)
- Drop is_nvidia_nouveau() helper. It is not called anywhere yet; can be re-added when a consumer lands. - Collapse the Sway+Nvidia confirmation result handling into a single expression now that allow_skip=False guarantees a boolean answer.
This commit is contained in:
parent
40cbc30997
commit
fc721cc94b
|
|
@ -81,17 +81,6 @@ class GfxDriver(Enum):
|
|||
case _:
|
||||
return False
|
||||
|
||||
def is_nvidia_nouveau(self) -> bool:
|
||||
"""
|
||||
True for the open-source nouveau driver (Mesa) for Nvidia GPUs.
|
||||
Currently only NvidiaOpenSource. Officially supported by Sway.
|
||||
"""
|
||||
match self:
|
||||
case GfxDriver.NvidiaOpenSource:
|
||||
return True
|
||||
case _:
|
||||
return False
|
||||
|
||||
def packages_text(self) -> str:
|
||||
pkg_names = [p.value for p in self.gfx_packages()]
|
||||
text = tr('Installed packages') + ':\n'
|
||||
|
|
|
|||
|
|
@ -105,8 +105,7 @@ class ProfileMenu(AbstractSubMenu[ProfileConfiguration]):
|
|||
preset=False,
|
||||
).show()
|
||||
|
||||
if not result.get_value():
|
||||
return preset
|
||||
return driver if result.get_value() else preset
|
||||
|
||||
return driver
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue