pagure

#813 hide the diff line numbers from on diff output
Closed by ryanlerch. Opened by ryanlerch.
ryanlerch/pagure remove_diff_linenos  into  master

Download 813.patch
no initial comment

This is an interium fix for issue #724, it just removes the line numbers from the diff output on the PR page. This is just done with CSS.

Ultimately to fix #724, we will need to fully rework the way diffs are presented to the user.

:-1: for me here, if we drop the line number we can no longer highlight a line or a section of a PR and send it to someone.

I agree that having the line number of the file would be nicer but in the mean while having the line number of the diff isn't really problematic imho.

Fair enough, going to close this one.

Pull-Request has been closed by ryanlerch
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.