old-balance-ios icon indicating copy to clipboard operation
old-balance-ios copied to clipboard

Use Monospace font for Address Labels

Open borwahs opened this issue 5 years ago • 12 comments

  • Set font for Address UILabels to monospace (Menlo).

borwahs avatar Mar 30 '19 17:03 borwahs

On it. Was debating whether to build an extension and change the existing ones.

borwahs avatar Mar 30 '19 18:03 borwahs

See comparison of pre and post Menlo usage below.

Couple of notes:

  • Menlo only has Bold and Regular for the font weight so I use Bold if medium/heavy/bold are set.
  • There is no "light" option so I default to Regular.

pre-new prepost2

borwahs avatar Mar 30 '19 18:03 borwahs

Thoughts on changing the CDP Modal Info view numbers to the monospace font?

borwahs avatar Mar 30 '19 18:03 borwahs

Totally agree @borwahs. There are lots of places where I want to use monospaced fonts and this really helps me see how to do it. Thanks!

I'll let Ben review this before merging.

ricburton avatar Mar 31 '19 07:03 ricburton

Really interesting discussion here: https://stackoverflow.com/questions/38148073/uifont-monospaceddigitsystemfontofsize-not-really-monospaced

In all the designs I use SF Mono but that is actually not allowed apart from on Apple's products.

I think we need to find a monospaced font that works well for the project.

I tweeted about this before. Will look at the suggestions.

ricburton avatar Mar 31 '19 09:03 ricburton

@ricburton // nice find on that SO question. I was confused why monospaced system don’t was not actually using it. Looks like the adjust font to size of container option breaks it.

borwahs avatar Mar 31 '19 16:03 borwahs

Yeah it’s really annoying they don’t make it available... we snuck it into balance for Mac and they approved it but they could reject at any time so it’s better to find a different one that’s fully allowed

einsteinx2 avatar Mar 31 '19 16:03 einsteinx2

Will hold off on merging this until we settle on the font we want.

einsteinx2 avatar Mar 31 '19 17:03 einsteinx2

Also for the name of the extension swift file, I accidentally wrote UIFont-Monospaced.swift when it should be UIFont+Monospaced.swift

einsteinx2 avatar Mar 31 '19 17:03 einsteinx2

I did something odd here. I think I might have not branched off of develop. Not sure why there are so many conflicts. Going to look into it and repush.

borwahs avatar Apr 02 '19 02:04 borwahs

fixed merge issues. think i accidentally rebased master at some point.

borwahs avatar Apr 02 '19 02:04 borwahs

@ricburton are you happy with Menlo as the font?

einsteinx2 avatar May 23 '19 03:05 einsteinx2