Skip to content

UI fixes for Mac and Linux#1580

Merged
cpaulgilman merged 13 commits into
developfrom
ui-mac-fixes
Nov 28, 2023
Merged

UI fixes for Mac and Linux#1580
cpaulgilman merged 13 commits into
developfrom
ui-mac-fixes

Conversation

@cpaulgilman

@cpaulgilman cpaulgilman commented Nov 16, 2023

Copy link
Copy Markdown
Collaborator
  • UI form fixes for Mac and Linux
  • Set width of tech list in configuration window and navigation menu in main window so long labels in collapsible trees are visible. This is a workaround, better solution would be to size tree width based on longest collapsed label.

@mjprilliman mjprilliman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, just a few nitpicks

Comment thread deploy/runtime/ui/Fuel Cell Costs Hybrid.json Outdated
Comment thread deploy/runtime/ui/Generic System Costs.json
Comment thread deploy/runtime/ui/MSLF System Control.json
Comment thread deploy/runtime/ui/Operating Costs Land Lease.json
@janinefreeman janinefreeman requested review from sjanzou and removed request for sjanzou November 17, 2023 23:51

@sjanzou sjanzou left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Smaller font looks good on the Mac
image

The funky scrolling through the top ribbon on the mac may be a separate issue
image

Forms look good
image

@cpaulgilman cpaulgilman merged commit 9f29894 into develop Nov 28, 2023
@cpaulgilman cpaulgilman deleted the ui-mac-fixes branch November 28, 2023 18:01
@cpaulgilman cpaulgilman added the added to release notes PR and/or issue has been added to release notes for a public release label Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

added to release notes PR and/or issue has been added to release notes for a public release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants