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

[MI-1987]: Integrated project list UI with: #14

Closed
wants to merge 53 commits into from
Closed

Conversation

avas27JTG
Copy link
Contributor

Integrated project list UI with:

  • Loader
  • Cards to show projects
  • Unlink confirmation dialog

Synced branches

image

image

image

@avas27JTG avas27JTG changed the title Mi 1987 [MI-1987]: Integrated project list UI with: Jul 28, 2022
height: 100%;
min-height: 350px;
display: flex;

Choose a reason for hiding this comment

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

Can use classes from utils.scss

webapp/src/components/emptyState/styles.scss Show resolved Hide resolved
type TooltipProps = {
tooltipContent: JSX.Element | string
children: JSX.Element
placement?: Placement,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
placement?: Placement,
placement?: Placement

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not required as eslint is not checking this

Comment on lines +8 to +9
text: string
slashCommand?: string
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't we create a type for this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this will be used just here so need to create a separate type

@avas27JTG avas27JTG changed the base branch from MI-1986 to MI-1986_1 August 11, 2022 06:02
@avas27JTG
Copy link
Contributor Author

Closing this PR and picked changes of this PR here Brightscout#28
this is done because syncing changes of this branch with the master is more expensive than creating a new PR

@avas27JTG avas27JTG closed this Aug 11, 2022
@avas27JTG avas27JTG deleted the MI-1987 branch August 11, 2022 06:45
avas27JTG pushed a commit that referenced this pull request Jan 6, 2023
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.

3 participants