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

Replace MessageArea with standard Gtk.InfoBar #1776

Merged
merged 4 commits into from
Jul 17, 2022

Conversation

infirit
Copy link
Contributor

@infirit infirit commented Jul 8, 2022

We don't really need one, Gtk.InfoBar has everything we need already. The rest is a small fix and rework.

infirit added 2 commits July 8, 2022 15:13
* Put ApplicationWindow in builder
* Use the Gtk.Grid for all layout
* Drop icon widgets and use icon names
The progressbar is lager than the stats and it would cause the box to
resize. Give it enough space by requesting at least 24px.
@infirit infirit force-pushed the dropmessagearea branch 3 times, most recently from a847748 to 324a549 Compare July 10, 2022 13:13
@infirit infirit marked this pull request as ready for review July 10, 2022 13:15
@infirit infirit requested a review from cschramm July 10, 2022 13:17
@infirit infirit force-pushed the dropmessagearea branch 2 times, most recently from 39c6690 to cab6e81 Compare July 10, 2022 14:16
cschramm
cschramm previously approved these changes Jul 11, 2022
blueman/main/Manager.py Outdated Show resolved Hide resolved
infirit added 2 commits July 14, 2022 18:01
A Gtk.InfoBar already has a Gtk.Revealer which animates perfectly fine for
our use.
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@infirit infirit requested a review from cschramm July 14, 2022 16:09
@infirit infirit merged commit 3fde657 into blueman-project:main Jul 17, 2022
cschramm added a commit to cschramm/blueman that referenced this pull request May 10, 2024
infirit pushed a commit to cschramm/blueman that referenced this pull request May 15, 2024
infirit pushed a commit to cschramm/blueman that referenced this pull request May 15, 2024
cschramm added a commit that referenced this pull request May 15, 2024
It was somehow dropped in #1776.
infirit pushed a commit to infirit/blueman that referenced this pull request May 15, 2024
infirit pushed a commit to infirit/blueman that referenced this pull request May 17, 2024
infirit pushed a commit to infirit/blueman that referenced this pull request Sep 19, 2024
infirit pushed a commit that referenced this pull request Sep 27, 2024
It was somehow dropped in #1776.
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