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

Minimap: "North" indicator arrow for circle minimap #3971

Closed
wants to merge 1 commit into from
Closed

Minimap: "North" indicator arrow for circle minimap #3971

wants to merge 1 commit into from

Conversation

sofar
Copy link
Contributor

@sofar sofar commented Apr 8, 2016

Replaces: #3730

This adds a simple, and small "North" indicator to the circular
minimap. The indicator is in a classical triangle-like arrow with a
little bit of shading to accentuate the shape and give it a little
bit depth. The indicator is stuck exactly at the edge as far outwards
as possible, and is not too intrusive but still easy enough to spot.

Related: #3730

This adds a simple, and small "North" indicator to the circular
minimap. The indicator is in a classical triangle-like arrow with a
little bit of shading to accentuate the shape and give it a little
bit depth. The indicator is stuck exactly at the edge as far outwards
as possible, and is not too intrusive but still easy enough to spot.
@sofar
Copy link
Contributor Author

sofar commented Apr 8, 2016

@nerzhul
Copy link
Member

nerzhul commented Apr 8, 2016

👍

5 similar comments
@Zeno-
Copy link
Contributor

Zeno- commented Apr 8, 2016

👍

@DonBatman
Copy link
Contributor

👍

@0-afflatus
Copy link

👍

@ObaniGemini
Copy link

👍

@indriApollo
Copy link

👍

@C1ffisme
Copy link

C1ffisme commented Apr 8, 2016

Wow. You actually found the (very simple, actually) way to make it multi-lingual!

👍

@paramat
Copy link
Contributor

paramat commented Apr 8, 2016

👍

@sfan5
Copy link
Member

sfan5 commented Apr 8, 2016

👍

@Fixer-007
Copy link
Contributor

Neutral %)

@kaadmy
Copy link

kaadmy commented Apr 8, 2016

👍

@kilbith
Copy link
Contributor

kilbith commented Apr 8, 2016

👊

@SmallJoker
Copy link
Member

👍

@nerzhul
Copy link
Member

nerzhul commented Apr 8, 2016

Merged af799d4

@nerzhul nerzhul closed this Apr 8, 2016
@PilzAdam
Copy link
Contributor

PilzAdam commented Apr 9, 2016

This is labeled incorrectly. It has four approvals, not two.

@ObaniGemini
Copy link

@PilzAdam : Then you should create 6 other labels :p

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.

None yet