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

Allow dragging the garrison mission UI frame #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lilyball
Copy link
Contributor

No description provided.

@tekkub
Copy link
Member

tekkub commented Dec 13, 2014

I'm curious why one would want to move this frame.

@lilyball
Copy link
Contributor Author

Because I can't see my full chat frame with this in the default position. It covers up the right third of my chat, meaning I can't actually read the output of "Complete All Missions".

It also helps when applying weapons/armor to followers, as it overlaps my bags in the default position.

@tekkub
Copy link
Member

tekkub commented Dec 14, 2014

I can't actually read the output of "Complete All Missions"

Fair point. I would like to add a visual report to the mission frame at some point. It bugs me quite a lot that blizzard seems to have abandoned their decade old standard of having visual and chat output for things in favor of a stupidly over-designed pretty animated pile of useless fluff. mumble

It also helps when applying weapons/armor to followers, as it overlaps my bags in the default position.

I want to get follower upgrading integrated into the main follower panel, actually. Are you using a bag addon? The one I use manages its frame level so that it comes to the front of the pile when needed.

@lilyball
Copy link
Contributor Author

I'm using a bag addon, but if the bags overlap the mission frame that bugs me as well.

@spiralofhope
Copy link

I use a bag addon, and I often open it and fiddle with previous mission rewards while I wait for the mission manager to complete new ones.

If my bags were overlapped by the mission frame, I'd be unable to do this.

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.

3 participants