Skip to content

Comments

Add workspace panel error boundary#3742

Open
ayolab wants to merge 3 commits intomainfrom
ayolab/add-workspace-panel-error-boundary
Open

Add workspace panel error boundary#3742
ayolab wants to merge 3 commits intomainfrom
ayolab/add-workspace-panel-error-boundary

Conversation

@ayolab
Copy link
Contributor

@ayolab ayolab commented Feb 17, 2026

No description provided.

@ayolab ayolab force-pushed the ayolab/add-workspace-panel-error-boundary branch from 8c444b0 to 30031ea Compare February 17, 2026 12:15
Signed-off-by: Ayoub LABIDI <ayoub.labidi@protonmail.com>
@ayolab ayolab force-pushed the ayolab/add-workspace-panel-error-boundary branch from 30031ea to 2041f1c Compare February 17, 2026 12:26
@sBouzols sBouzols self-requested a review February 25, 2026 09:01
return (
<Box sx={styles.container}>
<Box sx={styles.centeredContent}>
<ErrorOutlineIcon sx={styles.icon} />
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
<ErrorOutlineIcon sx={styles.icon} />
<ErrorOutlineIcon fontSize='large' color='error' />

and remove icon style ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

container: {
display: 'flex',
flexDirection: 'column',
height: '100%',
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
height: '100%',
flexGrow: 1,

?

Copy link
Contributor

Choose a reason for hiding this comment

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

And make the parent Box a flex container ?

Signed-off-by: Ayoub LABIDI <ayoub.labidi@protonmail.com>
@sonarqubecloud
Copy link

@ayolab ayolab requested a review from sBouzols February 25, 2026 10:29
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.

2 participants