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

Port to GTK4 #157

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

Port to GTK4 #157

wants to merge 25 commits into from

Conversation

treppenwitz03
Copy link
Contributor

@treppenwitz03 treppenwitz03 commented Dec 21, 2021

Initial port to GTK4.

Problem:

  • The notification window does not display anything but a transparent window
  • Handle motion signals

Blocked by:

  • Positioning is left to window manager

treppenwitz03 and others added 7 commits December 21, 2021 09:43
* Add CanberraGtk4 Namespace for Replacement of libcanberra-gtk

The only function that we was using from libcanberra-gtk was context_get(),
CanberraGtk4.context_get() reimplements it using the Gtk4 types.

* Update README and CI

* lint

* Make the code actually work
@treppenwitz03 treppenwitz03 marked this pull request as draft January 14, 2022 03:29
@danirabbit
Copy link
Member

@zenitsudev did you want to update this draft? :)

@treppenwitz03
Copy link
Contributor Author

sorry for the late reply. I am now free so I can work on this

@treppenwitz03 treppenwitz03 requested review from danirabbit, Marukesu and a team May 10, 2022 04:41
@treppenwitz03 treppenwitz03 marked this pull request as ready for review May 13, 2022 08:32
@treppenwitz03
Copy link
Contributor Author

On my testing, it works flawlessly now. However, there are signals where I commented some codes because I am not sure about what to do with them. The thing that maybe left to do as for now is to be able to position the notifications in their proper place.

@treppenwitz03
Copy link
Contributor Author

Goes with elementary/gala#1427

A window title is set as a unique identifier for the notification bubbles. It is hidden though, so it will not be visible and serve only as a name the window manager can anticipate so it can place the notification bubble in the proper area.

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

Successfully merging this pull request may close these issues.

3 participants