Skip to content
This repository was archived by the owner on May 30, 2024. It is now read-only.

Conversation

@noahingh
Copy link
Member

No description provided.

@noahingh noahingh added the component: ui This issue or pull request is related to the feature of the UI-side label Dec 19, 2021
@noahingh noahingh added this to the v0.5 milestone Dec 19, 2021
@noahingh noahingh linked an issue Dec 19, 2021 that may be closed by this pull request
<Popover title="Statuses" content={content}>
<Popover
title="Statuses"
trigger="click"
Copy link
Member Author

Choose a reason for hiding this comment

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

Change the trigger into click

Comment on lines +20 to +21
height: "200px",
overflow: "scroll"
Copy link
Member Author

Choose a reason for hiding this comment

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

Add height, overflow attributes to handle the overflow.

<Avatar size="small" src={status.avatarUrl} shape="square"/>
<Text
strong
ellipsis
Copy link
Member Author

Choose a reason for hiding this comment

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

Ellipse the overflowed text.

image

@noahingh noahingh merged commit 36cd7c4 into main Dec 20, 2021
@noahingh noahingh deleted the fix-overflow-commit-status branch December 20, 2021 12:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

component: ui This issue or pull request is related to the feature of the UI-side

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix the height of the popover of commit status to avoid overflow.

2 participants