Skip to content

Fix a lot of "TODO PORT" sections#1384

Merged
gsemet merged 4 commits into
Guake:masterfrom
aichingm:dev/todo-port
Aug 24, 2018
Merged

Fix a lot of "TODO PORT" sections#1384
gsemet merged 4 commits into
Guake:masterfrom
aichingm:dev/todo-port

Conversation

@aichingm

Copy link
Copy Markdown
Collaborator

There are still some TODO PORT sections left but I can't test stuff related to Unity and appindecator since I don't have a 16.-- Ubuntu box running...

I ported the monitor selection when using the "use_mouse" option to Gdk since the Gtk parts are deprecated, but some one should test them (I have currently only my laptop with me and no access to a second monitor)

I also changed https://github.com/Guake/guake/compare/master...aichingm:dev/todo-port?expand=1#diff-7a93ec37877f9e75dacff02a373a6734R39

and added a run-prefs make target to make development of the prefs window more enjoyable.

@gsemet

gsemet commented Aug 23, 2018

Copy link
Copy Markdown
Member

look wonderful ! I'll test dual monitoring later this week or next week, but if it is broken i'll fix it, do not worry.

Can you add a make reno SLUG=gtk3 about your work on all these ports? You can fill it in several commits when you submit your various pull requests !

@gsemet gsemet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

just the paths.py that should not be commited

Comment thread guake/paths.py Outdated
@@ -0,0 +1,98 @@
# -*- coding: utf-8; -*-

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

normally you do not commit this file is automatically generated by the makefile

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Of course, you are right!
I rebased and removed the file.

load default font via python Gio and not via cli call
test and add json example for custom commands
port screen selection (use mouse) to Gdk
add notification for failed show-hide key combinations settings
port one-click keyboard settings
port word char exceptions
replace Gtk.HBox with Gtk.Box+Gtk.Orientation.HORIZONTAL
@aichingm

Copy link
Copy Markdown
Collaborator Author

I have already done the rewrite of the notebook to allow for multiple terminals in one page, just waiting for this to be merged to keep the PRs small. It is not the most beautiful code jet since this is only step 3. and not all relevant functions are moved from guake_app.py to notebook.py but its a good start.

@gsemet gsemet merged commit d7674ba into Guake:master Aug 24, 2018
@gsemet

gsemet commented Aug 24, 2018

Copy link
Copy Markdown
Member

thanks a lot !

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