-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Render blob panel even if the highlighting request failed #11034
Conversation
This caused #10999 where no git history would show up, because the request for the highlighted content fails, as the file is binary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
@@ -240,6 +240,14 @@ export class BlobPage extends React.PureComponent<Props, State> { | |||
} | |||
repoHeaderContributionsLifecycleProps={this.props.repoHeaderContributionsLifecycleProps} | |||
/> | |||
<BlobPanel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it make no difference that this is now in the top of the DOM (before the file content) and not at the end?
Wish this component had screenshot tests...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It actually renders nothing, it's just a view provider wrapper. If that sounds safe enough, wanna approve? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very weird
Changelog entry pls :) |
This caused #10999 where no git history would show up, because the
request for the highlighted content fails, as the file is binary.
The Blob panel should be rendered independently of the blob content. That way
Closes #10999