Skip to content

OSX pink menu bug - #589

Merged
proletesseract merged 11 commits into
navcoin:masterfrom
proletesseract:521-osx-png-bug
Aug 21, 2019
Merged

proletesseract merged 11 commits into
navcoin:masterfrom
proletesseract:521-osx-png-bug

Conversation

@proletesseract

Copy link
Copy Markdown
Member

Fixes #521

Screen Shot 2019-08-20 at 9 01 14 PM

OSX build: https://builds.navcore.org/521-osx-png-bug

Gitian building the other OS binaries now to smoke test it doesn't break the layout on other operating systems.

rebasing to navcoin:master
rebasing to upstream master
rebasing to navcoin:master
merging master into 4.7.0
updated PNG's and changed border-image to background
adding bg colors
removing repeat-y
@aguycalled

Copy link
Copy Markdown
Member

image

image

@aguycalled

Copy link
Copy Markdown
Member

The menu icons show with an appreciable lower resolution, but this can be a temporary valid solution until the whole redesign is finished. Would be interesting to confirm how it shows in other OS/resolutions.

@proletesseract

Copy link
Copy Markdown
Member Author

@aguycalled yeah Juguelio reported the same, it works but its a bit blurry.

I have the other OS's compiled here now, so ill try to get people on discord to try it out:

https://builds.navcore.org/521-osx-png-bug/

@proletesseract

proletesseract commented Aug 20, 2019 •

Copy link
Copy Markdown
Member Author

Ubuntu:
unknown (5)

Windows 7:
unknown (6)

Courtesy of salmonskinroll

@proletesseract

Copy link
Copy Markdown
Member Author

and i agree, this is a quick interim fix before replacing the menu with @mxaddict's work. I could spend more time going down the pxmap approach but i dont think its worth the effort.

@mxaddict mxaddict left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

tested Gitian Build on windows 10, looks fine.

@proletesseract
proletesseract merged commit e087919 into navcoin:master Aug 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OSX gitian build of master branch is funky looking

3 participants