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

Blame parent link in blame view #2

Closed
sindresorhus opened this issue Feb 15, 2016 · 4 comments
Closed

Blame parent link in blame view #2

sindresorhus opened this issue Feb 15, 2016 · 4 comments

Comments

@sindresorhus
Copy link
Member

Often when I blame something there are useless noise commits obscuring what I'm actually looking for, which means I need to go one commit back from that commit that made the noise, and then blame again to get what I'm actually after. Would be useful to have a link to go one commit back in the blame view.

@sindresorhus sindresorhus changed the title Go to parent link in blame view Blame parent link in blame view Feb 15, 2016
@paulmolluzzo
Copy link
Contributor

Some sort of annotated screenshot would clarify this ticket a bit.

@sindresorhus
Copy link
Member Author

In the blame view, like https://github.com/sindresorhus/refined-github/blame/master/extension/content.js, I would like to to change the blame view to the parent commit of the one currently on the left side, as I want to know what happened to that piece of code before the commit currently there.

image

So in the above screenshot I would like an extra link for each blame that says Blame parent and goes to the blame view of the commit before the one referenced.

Is that clear?

@hkdobrev
Copy link
Contributor

I get exactly what you need, because I've wanted that for so long. I will submit a PR in a few minutes if everything goes by plan.

@sindresorhus
Copy link
Member Author

For reference, here's how it looks now:

napkin 26-02-16 1 58 44 pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants