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.

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

Feature/share popover #586

Merged
merged 7 commits into from
Nov 4, 2023
Merged

Feature/share popover #586

merged 7 commits into from
Nov 4, 2023

Conversation

dontry
Copy link
Contributor

@dontry dontry commented Oct 26, 2023

No description provided.

return (
<div className="share-panel">
<h3 style={{ marginTop: '4px' }}>Share the Diagram on Confluence<sup>*</sup></h3>
{isLoading ?
Copy link
Contributor

Choose a reason for hiding this comment

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

Loading 状态可不可以只影响Copy Link那个Button。现在这样影响整个Panel,给用户的感觉响应太慢。用户看到静态内容,然后Copy Link稍微等一下是可以的,因为用户的注意力还没有立刻转移到那里。

<br />
<div>
<p>Preview</p>
<div className="preview" >
Copy link
Contributor

Choose a reason for hiding this comment

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

Preview的设计请参考这个页面上的那个Card: https://zenuml-demo.atlassian.net/wiki/spaces/DEMO/pages/511049729/Embed+ZenUML.cn

@MrCoder MrCoder marked this pull request as draft October 27, 2023 01:54
@MrCoder
Copy link
Contributor

MrCoder commented Oct 31, 2023

@dontry Please resolve the conflict and update preview card according to https://github.com/ZenUml/web-app-laravel/pull/151

@MrCoder
Copy link
Contributor

MrCoder commented Oct 31, 2023

Once it is ready, please remove the "Draft" status.

@dontry
Copy link
Contributor Author

dontry commented Nov 2, 2023

@dontry Please resolve the conflict and update preview card according to https://github.com/ZenUml/web-app-laravel/pull/151

@MrCoder Can I remove all unecessary asyn/await from app.tsx file?

@MrCoder
Copy link
Contributor

MrCoder commented Nov 4, 2023

@MrCoder Can I remove all unecessary asyn/await from app.tsx file?
Sure

@MrCoder MrCoder marked this pull request as ready for review November 4, 2023 04:33
@MrCoder MrCoder merged commit ca4aba4 into master Nov 4, 2023
1 check passed
Copy link

github-actions bot commented Nov 4, 2023

Visit the preview URL for this PR (updated for commit 934de2b):

https://staging-zenuml-27954--pr586-feature-share-popove-h70n1rof.web.app

(expires Sat, 11 Nov 2023 04:36:43 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 6b46293b8a64352bd380930ba2a9594929b524e6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants