Skip to content

Conversation

@eloycoto
Copy link
Contributor

Small change on how to see the branch, is a bit cleaner for the end user:

image

@rhdh-gh-app
Copy link

rhdh-gh-app bot commented Feb 12, 2026

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @red-hat-developer-hub/backstage-plugin-x2a

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-x2a workspaces/x2a/plugins/x2a none v1.0.0

Copy link
Contributor

@elai-shalev elai-shalev left a comment

Choose a reason for hiding this comment

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

Cool!

@elai-shalev
Copy link
Contributor

elai-shalev commented Feb 12, 2026

I'll let @mareklibra merge if there is any UI edge case I'm not aware of

<br />
{url}
<div className={classes.root}>
<span className={classes.chip}>
Copy link
Member

Choose a reason for hiding this comment

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

It would be better to use https://v4.mui.com/components/chips/ to stay aligned with the rest of the Backstage.

}));

const GitBranchIcon = () => (
<SvgIcon fontSize="small" viewBox="0 0 24 24">
Copy link
Member

Choose a reason for hiding this comment

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

I can not find the icon in MUI, so this is good to use.

Copy link
Member

@mareklibra mareklibra left a comment

Choose a reason for hiding this comment

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

Better to use existing components if possible

@sonarqubecloud
Copy link

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