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

PartitionBar: make sure we get click events #765

Merged
merged 2 commits into from
May 8, 2024

Conversation

danirabbit
Copy link
Member

@danirabbit danirabbit commented May 7, 2024

Fixes #763
Closes #767

I can't actually get the menu to popup in this branch or any previous commit since the last release, but this makes sure the gesture sends the event at least. Maybe the menu only works in an actual installer session for some reason?

@danirabbit danirabbit marked this pull request as ready for review May 7, 2024 18:04
@danirabbit danirabbit requested a review from a team May 7, 2024 18:04
@vjr
Copy link
Member

vjr commented May 8, 2024

@danirabbit

When I click I see a message like "gtk popover - assertion failed - GTK_IS_POPOVER is not a popover"...

I had tried to fix this issue - I also changed Box to EventBox and also re-added the button pressed mask. I had to move the gesture.connect (menu.popup) from the construct block up to the constructor, because apparently the block was being executed before the constructor where the menu was actually being created - this fixed the debug message about "GTK_IS_POPOVER not being a popover".

I also re-added the local/private method that called menu.popup () with a log statement that confirmed the click event is happening but menu.popup () does nothing.

Is there some sort of GTK setup required - like adding the widgets in a hierarchy so that the event from the eventbox (partitionbar) is propagated down to the partitionmenu widget?

hope what ive written here is not too much gibberish :-)

@danirabbit
Copy link
Member Author

@vjr okay so this has multiple layers haha.

One layer is this: which means even when the menu does popup it's closed immediately #767

@vjr
Copy link
Member

vjr commented May 8, 2024

@danirabbit so how do i test? do i need to apply both PRs or just #767 ?

@danirabbit

This comment was marked as outdated.

@danirabbit danirabbit force-pushed the danirabbit/partitionbar-gesturefix branch from fe5b1ee to 1182aad Compare May 8, 2024 19:12
@danirabbit
Copy link
Member Author

@vjr okay actually, it was simpler than I thought and it's now fixed in this branch. This can be rebase merged :)

@danirabbit danirabbit force-pushed the danirabbit/partitionbar-gesturefix branch from 1182aad to e09eeac Compare May 8, 2024 19:15
Copy link
Member

@vjr vjr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - the popup is working now - please merge so the upcoming daily iso builds get this fix - i intend to install and further test os8 - thanks!

@danirabbit danirabbit force-pushed the danirabbit/partitionbar-gesturefix branch from e09eeac to 0d7d53d Compare May 8, 2024 20:19
@danirabbit danirabbit enabled auto-merge (rebase) May 8, 2024 20:19
@danirabbit danirabbit merged commit 151ec7f into master May 8, 2024
3 of 5 checks passed
@danirabbit danirabbit deleted the danirabbit/partitionbar-gesturefix branch May 8, 2024 20:20
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.

Unable to select partition in partition bar
2 participants