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

Font Size #24

Closed
kblairwhite opened this issue May 18, 2014 · 6 comments
Closed

Font Size #24

kblairwhite opened this issue May 18, 2014 · 6 comments

Comments

@kblairwhite
Copy link
Collaborator

*Add Plot Screen "Stake Orientation" label overlaps with tabbed bar.

I know that a lot of work was put into making font size larger, but I am just curious if this is necessary or if it was requested by stakeholders? I know that the text may be difficult to see in the field, so larger text may be required.

Personally I find the new text size a little chunky and tight to the input fields, as well all the stock buttons are still original size. (This is just my option tho.)

@NickGulajec
Copy link
Collaborator

When testing I noticed and fixed the add plot screen overlap just as you posted the issue :) I missed that .tss file last night.

The font size can be modified and was only part of the changes. Some other changes include:
-list screen rows now have a header height of 80 and row heights of 60.
-the Add label is sized to pick up clicks much easier and avoid accidental navigation presses.

There are UI fonts that cannot be modified easily. TabbedBar fonts are static, and it would take some work to recreate them with ButtonBars just to change the font size. Same thing with the Nav buttons, custom ones can be written, but the ones provided can't be modified AFAIK.

Because of these issues everyone's feedback is appreciated. The idea behind increasing the font was to increase readability but also to make use of the screen available. Seeing a screen with all the functionality crammed into the top quarter of the layout feels wrong.

Perhaps the font could be brought back down to 20 (from 24, issue #24) and two lines of text for each row could be used (bringing each row's height up to ~80) to make better use of the screen.

Thoughts?

edit: the TextField height can be increased to address the "tightness" of input fields, 36 works nice.

@GregTurner
Copy link

I'd to help if I can. Want to put a screen cap into this issue so I can see?

On Sunday, May 18, 2014, Nick Gulajec notifications@github.com wrote:

When testing I noticed and fixed the add plot screen overlap just as you
posted the issue :) I missed that .tss file last night.

The font size can be modified and was only part of the changes. Some other
changes include:
-list screen rows now have a header height of 80 and row heights of 60.
-the Add label is sized to pick up clicks much easier and avoid accidental
navigation presses.

There are UI fonts that cannot be modified easily. TabbedBar fonts are
static, and it would take some work to recreate them with ButtonBars just
to change the font size. Same thing with the Nav buttons, custom ones can
be written, but the ones provided can't be modified AFAIK.

Because of these issues everyone's feedback is appreciated. The idea
behind increasing the font was to increase readability but also to make use
of the screen available. Seeing a screen with all the functionality crammed
into the top quarter of the layout feels wrong.

Perhaps the font could be brought back down to 20 (from 24, issue #24#24)
and two lines of text for each row could be used (bringing each row's
height up to ~80) to make better use of the screen.

Thoughts?


Reply to this email directly or view it on GitHubhttps://github.com//issues/24#issuecomment-43450126
.

@NickGulajec
Copy link
Collaborator

Here is the 'index' screen, site surveys, with 24 size fonts:
sitesurveys24

Here is the site survey screen modified with 20 size fonts and a row height of 48 (what the application has been set to for the past month):
sitesurvey20

And here is 20pt font with a row height of 60 for fun:
sitesurvey20 60

Note for the last two images I did not revert the header and footer heights to what they used to be.

@NickGulajec
Copy link
Collaborator

Those look alright. Where it starts looking cramped is AddTransect 24 font:
addtransect24

Here is what it used to look like in 20 font:
addtransect20

The editable TextFields can be resized to give the font inside more breathing room, like in this modal:
36heighttextfield
Note that the vertical distance between rows in this last shot has not be increased to accommodate the larger font and would need to happen.

@GregTurner
Copy link

You could make a couple graphical buttons. Imagine a square with each of
the corners filled in for top left bottom right. Every touch toggles the
graphic. There should be an image button control somewhere in the toolkit.

On Monday, May 19, 2014, Nick Gulajec notifications@github.com wrote:

Those look alright. Where it starts looking cramped is AddTransect 24 font:
[image: addtransect24]https://cloud.githubusercontent.com/assets/4177568/3016782/f4719f6e-df71-11e3-9722-02bdd1e61f90.png

Here is what it used to look like in 20 font:
[image: addtransect20]https://cloud.githubusercontent.com/assets/4177568/3016785/fce5a3e8-df71-11e3-8439-064c36163e15.png

The editable TextFields can be resized to give the font inside more
breathing room, like in this modal:
[image: 36heighttextfield]https://cloud.githubusercontent.com/assets/4177568/3016790/18336090-df72-11e3-82b8-0a03195153ff.png
Note that the vertical distance between rows in this last shot has not be
increased to accommodate the larger font and would need to happen.


Reply to this email directly or view it on GitHubhttps://github.com//issues/24#issuecomment-43525878
.

@NickGulajec
Copy link
Collaborator

The experiment with a larger font has concluded and all screens been reverted to fontSize:20. Other ways of utilizing screen real-estate are being explored.

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

No branches or pull requests

3 participants