manjaro / mdd

Manjaro Data Donor - WIP
MIT License
12 stars 3 forks source link

Add PySide6 GUI for MDD configuration #18

Closed fhdk closed 1 week ago

fhdk commented 1 week ago

Introduced a GUI for Manjaro Data Donor using PySide6. Updated requirements.txt to include PySide6 and expanded mdd.py with GUI components to set options for telemetry and scheduling. Additionally, implemented config file handling and enhanced service configuration methods.

yochananmarqos commented 1 week ago

Looks good so far. The scrollbars look a little out of place on GNOME, however.

Using QAdwaitaDecorations with Noto Sans 11:

Screenshot From 2024-11-10 12-36-44

Using the default Kvantum KvLibadwaitaMaiaDark theme with DejaVu LGC Sans 12:

Screenshot From 2024-11-10 12-40-09

EDIT: Added font and size info

fhdk commented 1 week ago

I had hardcoded the background and font color of the preview - that is what created the weird scrollbars.

I have removed that - then reorder the elements and shortend their names - leaving space for larger system font.

@yochananmarqos could you please test how it works on Gnome?

image

Hmm - I didn't see that before - now the layout seems out of balance ...

yochananmarqos commented 1 week ago

I installed to site-packages, but mddgui complains:

Traceback (most recent call last):
  File "/usr/bin/mddgui", line 271, in <module>
    run()
  File "/usr/bin/mddgui", line 262, in run
    mdd.prepare_inxi()
    ^^^^^^^^^^^^^^^^
AttributeError: module 'mdd' has no attribute 'prepare_inxi'
fhdk commented 1 week ago

I suspect the problem is due to the files is lacking the .py extension - the import does not work.

@romangg does not want the GUI to be part of mdd - but a stand alone script - that is why I moved the GUI to a separate file.

The files - if bundled together a different packaging needs to be done in /usr/lib/python3.12/site-packages and a symlink to mdd.py as /usr/bin/mdd

I will close this PR.

yochananmarqos commented 1 week ago

Ah, yes it was the file extensions. :man_facepalming:

With Kvantum, the scrollbars are only visible upon hover.

QAdwaitaDecorations: image

Kvantum: Screenshot From 2024-11-12 09-17-55

yochananmarqos commented 1 week ago

@fhdk By the way, I do not understand why you closed this. :thinking:

fhdk commented 1 week ago

@fhdk By the way, I do not understand why you closed this

Private feedback from @romangg - also mentioned in out Team Sync

https://github.com/fhdk/mdd/tree/feat-donate-gui