Move UI to react #25

CopyDemon · 2023-08-03T17:47:54Z

Based on the new design file, rewrite the UI with React:

Link to design file

The new UI should be broken down into components to ensure it's easy to maintain in the future.
The new UI should retain the functionalities of the old UI.

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

I agree my contributions are submitted under the license terms described in the LICENSE.txt file at the top level of this directory.
I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

… back later

…king

dangunter · 2023-09-13T14:57:42Z

It now works!
One minor thing: the IDAES logo in the upper-left corner is not found. Maybe the path needs to be relative to the source code not the run directory?

… more writing to file to share variables

…link image, let webpack to handle path when run build

…n highlight event to them

…and remove comment out old code

dangunter

I see some places in paper.js where 'var' is used instead of 'let' or 'const'. Could you go through and see if you can replace those? There are very few places var is still needed I think.

dangunter

LGTM

CopyDemon added 30 commits July 16, 2023 12:09

delete because rename the file

ce8153f

create basic header css

f8a1cb3

add context and header use context

76e0f8e

basic header flowsheet name area css

0c63add

create header flowsheet name component

0265d65

create header function btn component

72e13c5

basic style header function btn component

eff5b0f

create header function btn refresh

231dbb0

create header function btn save

00138cb

basic style for header function btn view and its drop down list

819fa6b

create component header function btn view and its dropdown

86f7ecb

style header logo

8ef7893

create component header logo component

3a1f7ca

create table compoent

240cbbe

create context for whole app, add panel show hide

ea0ac35

create interface for ts panelStateInterface

df96c13

add demoflowsheet.jsonOC

48caba7

install axio

2767e8c

move lib folder and files from old ui

9f1fc95

install jointjs

1800fa1

hack way to include jointjs with old lib rappid.tgz is not free check…

7395a35

… back later

disable stop error when build when has variable not use

f363f92

update demo flowsheet JSON

4751569

pass demo flowsheet json through context

12972ea

move demo flowsheet json to public

9d64533

ignore this please, this is only for testing joint js hack way is wor…

acdfaae

…king

copy icons from old ui

5119e32

fix, should call flowsheet state not demoFlowsheetState

22f738e

update flowsheet data

1e2ee58

remove joint test from app.tsx

ef4ee00

CopyDemon added 4 commits September 12, 2023 11:55

run build to dist

779f8f6

update indent

5dbd185

update panel always render but display control show and hide

f5ed2de

fetch and sync from main, and fix conflicts

8f64753

CopyDemon added 6 commits September 13, 2023 09:40

clean up create_shared_JSON fn left over, this function is removed no…

2c6a6ea

… more writing to file to share variables

use import not path to link to image

ed7e300

move image out public folder to src/assets/imags, then use import to …

18f1ee4

…link image, let webpack to handle path when run build

run build to dist

1bfbacb

fix cypress config

a94c4b0

format, run on black format 23

f2f730c

CopyDemon marked this pull request as ready for review September 21, 2023 18:36

CopyDemon added 4 commits September 28, 2023 09:45

update default size to 35 prevent default size over 100 error

6558b93

run build to dist

0722bf3

add condation to check only when fv and stream table exist then assig…

43b9065

…n highlight event to them

run build to dist

673740f

CopyDemon changed the base branch from move-ui-to-react to main September 28, 2023 18:00

ksbeattie requested a review from dangunter September 28, 2023 18:34

CopyDemon added 2 commits September 28, 2023 11:49

this is only clean up the code, add descriptions to functions, files …

60829b8

…and remove comment out old code

build to dist

468d93e

ksbeattie added the Priority:High High Priority Issue or PR label Sep 28, 2023

dangunter reviewed Sep 29, 2023

View reviewed changes

CopyDemon added 2 commits October 1, 2023 19:29

replace var with let

eb968c2

build to dist

48d26b1

dangunter self-requested a review October 6, 2023 17:41

dangunter approved these changes Oct 6, 2023

View reviewed changes

dangunter merged commit 372c3ff into IDAES:main Oct 6, 2023
17 checks passed

This was referenced Jul 11, 2024

Port flowsheet diagram into a react component #30

Closed

Fix label style #31

Closed

Break down FV generate classes to react component, no more classes #33

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Move UI to react #25

Move UI to react #25

CopyDemon commented Aug 3, 2023

dangunter commented Sep 13, 2023

dangunter left a comment

dangunter left a comment

Move UI to react #25

Move UI to react #25

Conversation

CopyDemon commented Aug 3, 2023

Legal Acknowledgement

dangunter commented Sep 13, 2023

dangunter left a comment

Choose a reason for hiding this comment

dangunter left a comment

Choose a reason for hiding this comment