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 Download Bar #34

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

Changed Download Bar #34

wants to merge 1 commit into from

Conversation

micah686
Copy link

Uploading new_download_bar.PNG . . .

Re-centered the downloads text, and moved the download progress bar down, while decreasing its height

Re-centered the downloads text, and moved the download progress bar down, while decreasing its height
@CaptainMagma
Copy link
Contributor

Could you please provide a before and after picture and a reason for these changes? It would help a lot.

@micah686
Copy link
Author

micah686 commented Apr 2, 2015

old download bar
new_download_bar
Since the "X Online" button isn't at the bottom right anymore, I decided to recenter the downloading text. Also, I think the changes keep more to the pixelvision skin feel than the previous one.

@CaptainMagma
Copy link
Contributor

Oh, I see what you mean. Although I don't disagree that it's a good idea, I don't think any of us (somini, mr. late, and myself) would agree to changing the way the skin looks from the way Pulseh had it in the first place. That being said, there's nothing stopping you from keeping it as your own personal mod since that file doesn't seem like one that gets updated often (last update for the uistatuspanel.layout was 11/11/2014).

Edit: On second thought, I don't think that's a very good reason to shoot down your idea, nor is it fair. Give me about a day and I'll think about it. Somini, I would like your opinion on this as well if you have one.

Edit2: Actually, I see now why this would not work. First off, the bottom portion of the screen does not really give you enough room to stack the download text on top of the download bar. It just looks a little too squished together, in my opinion, which makes it look a little less organized. Second, the thin black border of the download bar does not surround the blue bar entirely, but instead the bottom border runs off to the left a little too much (this is just a small problem that could probably be fixed easily).

@Efreak
Copy link
Contributor

Efreak commented Apr 5, 2015

I think it looks decent enough, if you can fix the issue with it. If you're worried about changing the skin from what Pulseh wanted, you could also add it as a tweak that's disabled by default.

PS, you can prefix a username with @ (ie @CaptainMagma), and GitHub will linkify the username, and also tell the user they got mentioned.

@somini
Copy link
Owner

somini commented Apr 6, 2015

Yeah, I think making this a tweak, or even an Override, is the way to go.
I'll investigate this further when I got the time.

@somini somini self-assigned this Apr 6, 2015
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.

4 participants