Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

update overview layout #3046

Merged
merged 7 commits into from
Nov 16, 2020
Merged

update overview layout #3046

merged 7 commits into from
Nov 16, 2020

Conversation

Lijiaoa
Copy link
Contributor

@Lijiaoa Lijiaoa commented Oct 29, 2020

TODO

  • table滚动条,固定表头
    固定一屏幕

related issue: #2937

image

init:
image

@Lijiaoa Lijiaoa marked this pull request as draft October 30, 2020 01:48
@liuzhe-lz liuzhe-lz mentioned this pull request Oct 30, 2020
77 tasks
@Lijiaoa Lijiaoa marked this pull request as ready for review November 6, 2020 02:16
@Lijiaoa
Copy link
Contributor Author

Lijiaoa commented Nov 6, 2020

fixed:
image

<Accuracy
accuracyData={accuracyGraphData}
accNodata={noDataMessage}
height={290}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it possible to move the styles into the css file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

</p>
</div>
<div className={styles.inner}>
<p className={styles.subtext} id={descriptionId} style={{ color: '#333' }}>
Copy link
Contributor

Choose a reason for hiding this comment

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

move the color to style file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

@liuzhe-lz liuzhe-lz merged commit 4244d85 into microsoft:master Nov 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants