UI refactoring (part 1) #1092

Merged
g1itch merged 20 commits from ui-refactoring into v0.6 2018-01-30 17:32:38 +01:00
g1itch commented 2018-01-11 15:41:14 +01:00 (Migrated from github.com)

Hello!

Here is the first part of UI refactoring. What has been done?

  • most dialogs moved into dialogs module;
  • made common dialogs code reusable;
  • slightly edited about.ui (I think the wiki page image is also needed here);
  • copyright year made smart.

As usual I ready to drop or squash any commit after review.

Hello! Here is the first part of UI refactoring. What has been done? * most dialogs moved into `dialogs` module; * made common dialogs code reusable; * slightly edited `about.ui` (I think the [wiki page image](https://bitmessage.org/w/images/f/f1/Bitmessagelogo-reduced.png) is also needed here); * copyright year made smart. As usual I ready to drop or squash any commit after review.
PeterSurda commented 2018-01-11 16:09:28 +01:00 (Migrated from github.com)

Thank you, this is the type of work that is necessary but I don't like doing it myself. I'll review it soon but it's probably ok.

Thank you, this is the type of work that is necessary but I don't like doing it myself. I'll review it soon but it's probably ok.
g1itch commented 2018-01-23 20:51:48 +01:00 (Migrated from github.com)

Oops, I have broken EmailGatewayDialog. dialog.exec_() not returns account.

Oops, I have broken `EmailGatewayDialog`. `dialog.exec_()` not returns account.
PeterSurda commented 2018-01-25 03:18:39 +01:00 (Migrated from github.com)

I'm reviewing this now, please no more commits in this PR unless it's a bugfix.

I'm reviewing this now, please no more commits in this PR unless it's a bugfix.
g1itch commented 2018-01-25 08:38:32 +01:00 (Migrated from github.com)

OK, that one was last. EmailGatewayDialog didn't close on one of my computers without self.hide().

OK, that one was last. EmailGatewayDialog didn't close on one of my computers without self.hide().
PeterSurda commented 2018-01-30 15:52:31 +01:00 (Migrated from github.com)

There's now a conflict with the offline menu, please resolve. I haven't had a time for a full review yet but I'm working on it.

There's now a conflict with the offline menu, please resolve. I haven't had a time for a full review yet but I'm working on it.
g1itch commented 2018-01-30 16:39:39 +01:00 (Migrated from github.com)

Done rebase.

Done rebase.
PeterSurda commented 2018-01-30 17:33:13 +01:00 (Migrated from github.com)

Thank you for this huge amount of work.

Thank you for this huge amount of work.
g1itch commented 2018-01-30 17:39:04 +01:00 (Migrated from github.com)

Now you may probably close #1081, #1040, #1089. #527 maybe remain as guideline.

Now you may probably close #1081, #1040, #1089. #527 maybe remain as guideline.
PeterSurda commented 2018-01-30 17:49:48 +01:00 (Migrated from github.com)

Done

Done
This repo is archived. You cannot comment on pull requests.
No description provided.