Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update my_application.cc #279

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Update my_application.cc #279

wants to merge 5 commits into from

Conversation

s-b-repo
Copy link

Key Changes:

KDE Detection: The code now checks for KDE's window manager, KWin, to avoid using a GNOME-style header bar when the app is running in KDE.
Window Title Bar: If not running in GNOME or KDE, the app will use a traditional window title bar instead of the GNOME header bar.

@s-b-repo
Copy link
Author

Header Comments: Added more detailed comments to explain what the MyApplication struct is and what the my_application_new() function does. This makes the file more informative for future maintainers or anyone unfamiliar with the code.

Documentation of Return Value: Added (transfer full) annotation in the function documentation, which follows the GObject Introspection conventions. This indicates that the caller takes ownership of the returned value and is responsible for freeing it.

General Formatting: Improved spacing and alignment for readability. The header guards and function prototypes are easier to read and maintain.

@NANI-SORE
Copy link
Collaborator

Please move build instructions to a separate file (something like build_linux.md). Since we stopped doing desktop builds for now we don't need linux build instructions in main readme

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants