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

Changed main toolbar to be flush with the rest of the window. #761

Merged
merged 8 commits into from Aug 12, 2014
Merged

Changed main toolbar to be flush with the rest of the window. #761

merged 8 commits into from Aug 12, 2014

Conversation

archshift
Copy link
Contributor

Tested on OS X, Linux, and Windows, but could probably use more testing.

Before, after (respectively):

OS X

Linux Mint

Windows 8

@shuffle2
Copy link
Contributor

shuffle2 commented Aug 8, 2014

Looks nice, how does debugger look?
@dolphin-emu-bot rebuild

@archshift
Copy link
Contributor Author

I've fixed the errors that made the Linux build fail (hopefully).

As for the debugger, I'm taking a good look at it now. As of this moment, the two drop-down buttons are on their own ugly-toolbar, which I'm looking to fix.

@OussamaDanba
Copy link
Contributor

Is it intentional that there is a short fade-in and fade-out when hovering over the toolbar buttons? The rest of the buttons don't have this behaviour. Personally I prefer the old behaviour.

Also, when pressing the Play/Pause button the toolbar changes width whereas previously the width would stay the same. The toolbar width should be fixed in my opinion.

Finally, I'm using windows. Looks pretty good apart from the mentioned remarks.

@shuffle2
Copy link
Contributor

shuffle2 commented Aug 8, 2014

I think the fading looks good, just personal opinion though...but would
probably look better if there were no tooltips popping up for the menubar
items (or maybe just took longer to trigger). The fading is also consistent
with animation of other highlights which you probably don't even notice,
like hovering over clickable buttons, tabs, etc. in other dolphin dialogs.

On Fri, Aug 8, 2014 at 3:26 PM, Oussama Danba notifications@github.com
wrote:

Is it intentional that there is a short fade-in and fade-out when hovering
over the toolbar buttons? The rest of the buttons don't have this
behaviour. Personally I prefer the old behaviour.

Also, when pressing the Play/Pause button the toolbar changes width
whereas previously the width would stay the same. The toolbar width should
be fixed in my opinion.

Finally, I'm using windows. Looks pretty good apart from the mentioned
remarks.


Reply to this email directly or view it on GitHub
#761 (comment).

@archshift
Copy link
Contributor Author

So I've integrated the drop-down buttons, now I need to find a way to re-enable their dropdown function :/

@shuffle2
Copy link
Contributor

shuffle2 commented Aug 8, 2014

@dolphin-emu-bot rebuild
To see what changes look like..

@archshift
Copy link
Contributor Author

I'm considering moving the perspective buttons to the menu. What do you think?

@shuffle2
Copy link
Contributor

shuffle2 commented Aug 8, 2014

SGTM. To be honest I don't know if the perspective feature even works. For sure it's not intuitive/easy to use.

Also, there are wxwidgets asserts firing when launching debugger.

@lioncash
Copy link
Member

lioncash commented Aug 8, 2014

@shuffle2 The last time I accidentally clicked on edit for perspectives it seemed to work. Does anyone even really use perspectives?

@archshift
Copy link
Contributor Author

Yeah, I'm reverting that entire last commit I made, which broke the perspectives. From there, I think I'll move all that stuff into the menubar.

@shuffle2
Copy link
Contributor

shuffle2 commented Aug 8, 2014

@lioncash well, in theory they are very nice...maybe it does work now, idk. I just know that when I was using dolphin's debugger heavily, they didn't work properly, so I'd have to rearrange the AUI windows every time a debugger session started :/

@lioncash
Copy link
Member

lioncash commented Aug 8, 2014

@shuffle2 Ah ok.

@archshift
Copy link
Contributor Author

It should be working fine on OS X and Linux, for the most part. I ran into an issue on Linux where the play button refuses to change its text to "pause", which doesn't happen on OS X or Windows. It should be able to compile with the buildbot at this point, and now I'm going to test on Windows.

@archshift
Copy link
Contributor Author

I've removed the commit that was reverted (and the revert commit as well) using git rebase -i. As it turns out, the pause button issue does not happen on Windows, only Linux indeed.

@degasus
Copy link
Member

degasus commented Aug 9, 2014

@dolphin-emu-bot rebuild

@archshift
Copy link
Contributor Author

This fixes issue 6351.

@shuffle2
Copy link
Contributor

lgtm

@lioncash
Copy link
Member

looks good to me

shuffle2 added a commit that referenced this pull request Aug 12, 2014
Changed main toolbar to be flush with the rest of the window.
@shuffle2 shuffle2 merged commit 0b27e0f into dolphin-emu:master Aug 12, 2014
@archshift archshift deleted the better-toolbar branch August 12, 2014 03:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants