CenterForTheBuiltEnvironment / comfort-dash

Revision of the CBE Comfort Tool using Dash.
2 stars 12 forks source link

New page #19

Closed Xuhui-Wang-9988 closed 2 months ago

Xuhui-Wang-9988 commented 2 months ago

Page Changes Description

Description

This document outlines the changes made to page structure and functionalities. It includes all the changes made during week 4 to week 5 that have not been merged into the main. It provides a detailed comparison of the modifications, highlighting new features, improvements, and any potential breaking changes. This description is based on the comparison of new page and main, which contains a comprehensive diff of the alterations.

Types of change

image image

  1. customised drop-down charts from my_config_file and set-up text and other contents image This is the example code block for changes:

image

  1. Option for drop-down selection This is the example code block for what we have done image and it shown on page as:

image

  1. changes on footer to fit the mobile view

image Addtioinaly, as shown on the navibar, the top-left icon has not been decided, we now used the basic icon of the CBE project

  1. change of navibar to fit the mobile view image

  2. change on footer and navibar to add hyperlink

This is example code block from navibar file: image it shown on page as: image The icon of GitHub can jump to the project repository once clicked.

  1. Edit texts on footer and navibar image image
  1. changes of code blocks for formal coding image

How Has This Been Tested?

The page changes have been tested through the following methods:

  1. Manual testing of each modified component
  2. Integration testing to ensure new features work correctly with existing functionality
  3. Cross-browser testing for compatibility
  4. Responsive design testing on various device sizes

Checklist

For a complete understanding of the changes, it is recommended to review the push request in conjunction with this description.

FedericoTartarini commented 2 months ago

Thank you all very much for working on this. I am extremely happy to see that all this work has been done.

Unfortunately, it is almost impossible to properly review the code since there are a lot of commits and most of them are related to code formatting. @t-kramer shall we accept the pull request and then review the code separately?

t-kramer commented 2 months ago

Thank you all very much for working on this. I am extremely happy to see that all this work has been done.

Unfortunately, it is almost impossible to properly review the code since there are a lot of commits and most of them are related to code formatting. @t-kramer shall we accept the pull request and then review the code separately?

Sounds good to me.