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.

Sign up for GitHub

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

Jump to bottom

Morph base app UI to resemble final product more closely #86

Merged
merged 15 commits into from
Oct 15, 2019
Merged

Morph base app UI to resemble final product more closely #86

merged 15 commits into from
Oct 15, 2019

Conversation

Aulud
Copy link

@Aulud Aulud commented Oct 15, 2019 •
edited

Morph base app UI to resemble final product more closely

Summary: Implements changes to the basic UI of AB3 to set the canvas for subsequent design of GUI elements for new commands. Addresses #60. This supersedes PR #54, which was merged erroneously earlier.

Changelog

  • Resize app dimensions to 384px (H) by 700px (H) for mobile form factor
  • Suppress resizing of app window; force default window size on startup regardless of GUI preferences
  • Redesign and style base GUI of app component-wise to a light theme
    • Remove redundant header menu and all references to menu UI elements
    • Reduce font sizes; force result display label to wrap text
  • Separate CSS stylesheets by category into one of LightTheme.css, Components.css or Extensions.css

Screenshots

Before-after comparison

Splitwiser-comparison

Last updated 15 Oct 2019, 11:00PM

@Aulud Aulud requested a review from a team October 15, 2019 09:04
daekoon
daekoon previously approved these changes Oct 15, 2019
View reviewed changes
Copy link

@daekoon daekoon left a comment

Choose a reason for hiding this comment

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

LGTM

@Aulud Aulud requested a review from a team October 15, 2019 15:07
daekoon
daekoon approved these changes Oct 15, 2019
View reviewed changes
Copy link

@daekoon daekoon left a comment

Choose a reason for hiding this comment

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

LGTM

@daekoon daekoon merged commit 688dd51 into master Oct 15, 2019
@daekoon daekoon deleted the morph-app-base-ui branch October 15, 2019 15:58
This was referenced Oct 16, 2019
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daekoon daekoon daekoon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Aulud @daekoon