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

Add indicator to inform users about execution of 'heavy' operations #255

Closed
AndreiMazol opened this issue Jan 19, 2021 · 2 comments · Fixed by #288
Closed

Add indicator to inform users about execution of 'heavy' operations #255

AndreiMazol opened this issue Jan 19, 2021 · 2 comments · Fixed by #288
Assignees
@softcat23 @AndreiMazol
Labels
enhancement

Comments

@AndreiMazol
Copy link
Contributor

AndreiMazol commented Jan 19, 2021 •
edited
Loading

It is necessary to show loading indicator to inform users about execution of the following operations:

  • clean up
  • layout
  • aromatize
  • dearomatize
  • calculate CIP
@AndreiMazol AndreiMazol removed the draft label Jan 27, 2021
@Ksanjara
Copy link

  • During the operation listed above buttons should be disabled.
  • Canvas should be disabled and spinner should displays on the center of canvas.
    Prototype:
    Loading

@softcat23 softcat23 self-assigned this Feb 1, 2021
softcat23 added a commit that referenced this issue Feb 4, 2021
@softcat23
Merge branch 'master' into #255-progress-indicator
307987b
@softcat23 softcat23 linked a pull request Feb 4, 2021 that will close this issue
#255 Add indicator to inform users about execution of 'heavy' operations #288
Merged
softcat23 added a commit that referenced this issue Feb 5, 2021
@softcat23
Merge branch 'master' into #255-progress-indicator
e57a958
softcat23 added a commit that referenced this issue Feb 8, 2021
@softcat23
Merge branch 'master' into #255-progress-indicator
f2433f3
AndreiMazol pushed a commit that referenced this issue Feb 8, 2021
@softcat23
#255 Add indicator to inform users about execution of 'heavy' operati…
708a370 
…ons (#288)

* add new action and reducer

* refactor request reducer

* disable buttons on indigo request

* reorganize editor; add rough spinner component

* implement spinner component

* add reducer test

* fix failing index.test.tsx

* convert test to ts file

* revert

* fix mapDispatchToProps function

* reorganize folders

* add copyright and fix year in the changed files

* move disabled logic in the variable assignment

* fix gengroup styles
@Ksanjara
Copy link

Ksanjara commented Feb 8, 2021

Issue is verified and closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@softcat23 softcat23

@AndreiMazol AndreiMazol

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#255 Add indicator to inform users about execution of 'heavy' operations epam/ketcher
4 participants
@AlexanderSavelyev @softcat23 @AndreiMazol @Ksanjara