[Feature Request] Formatting for CSV Diff

[kdumontnu]

We should apply the same styling from CSV file-view to diffs for CSVs.

It should looks something like this:

[kdumontnu]

Bounty available!
https://www.bountysource.com/issues/95872476-feature-request-formatting-for-csv-diff

Requirements:

• Apply the same or similar styling as file-view.tmpl: file-view csv markdown to CSV files in the diff view
• Add a toggle button to load code vs. render (shown above) with tooltips

Requests:

• Add summary for "# rows added/deleted, # columns added/deleted" in addition to overall "changes", which already is supported by Gitea
• If entire row is modified or added/deleted, render the diff left/right instead of top/bottom (see example below)

[lunny]

But if the diff is to add one column or delete one column. Then every line changed?

[techknowlogick]

But if the diff is to add one column or delete one column. Then every line changed?

In that case currently it'd show a diff of each line changing, with this ticket it would show in a similar fashion but in a nice formatted way.

[kdumontnu]

But if the diff is to add one column or delete one column. Then every line changed?

In that case currently it'd show a diff of each line changing, with this ticket it would show in a similar fashion but in a nice formatted way.

Oh, that's a good question. I think techknologick is right that in the case where an entire row or column is deleted no special case is needed. Also, this should probably use the same config threshold for "diff too large to display" as any other text file.

However, you gave me an idea I'll add above: it would be nice to add a summary at the top: ex. "+3 rows, -2 columns". This all assumes that there are nice, properly-formatted csv files.

I think the other opportunity for improved styling is if an entire column is modified. In that case it would probably be better to show the "diff" to the right, as shown below. However, this is a "nice-to-have" and should not gate merging a PR for this. We can create a separate issue if needed.

[thisispalash]

Hi, I am interested in working on this issue. However, I searched the files on your code base for file-view.tmpl which does not exist. I think the project is doable, so once I know how you want your view, I can get started. Lastly, I want to say a week or so to complete as I also have schoolwork, but I don't think it should be more than that. Let me know if this issue [and bounty] is still open.

[kdumontnu]

Hi, I am interested in working on this issue. However, I searched the files on your code base for file-view.tmpl which does not exist. I think the project is doable, so once I know how you want your view, I can get started. Lastly, I want to say a week or so to complete as I also have schoolwork, but I don't think it should be more than that. Let me know if this issue [and bounty] is still open.

Not sure what you mean by searching the files on my code-base, but as far as I know no one has started working on this & bounty is still available! If you're going to work on this please fork from this upstream repo, not my fork.

In order to complete this issue/bounty you'll need to get a PR approved and merged to gitea master. For that you'll have to follow the contribution guidelines & in this case most likely add some unit/integration tests. If you'd like some help getting test cases together I can help.

Good luck - let me know if you have any other questions.

[thisispalash]

I meant looking for the file file-view.tmpl to understand better what you guys wanted.

I forked it from this branch and will take a look.

Lastly, yeah I think I will need some guidance in testing but I shall contact you then.

[KN4CK3R]

I have implemented this feature and improved the rendering a little bit. Currently the toggle button is missing but that should be easy compared to the csv diff.

A simple change:

Changes in different areas (note the jumping line numbers):

A removed column is rendered without marking all rows as changed:

[KN4CK3R]

How could the render/source toggle be implemented? Currently I use a display=source query parameter. This works fine for a single file. But if you have a diff with multiple files this would toggle all of them and not just the selected file.

[kdumontnu]

How could the render/source toggle be implemented? Currently I use a display=source query parameter. This works fine for a single file. But if you have a diff with multiple files this would toggle all of them and not just the selected file.

Yeah, imo this is an issue with how gitea relies heavily on dynamic serving. I believe you will need to rely on some javascript to toggle the styling on the client side. IMO, this is a prime example of where a proper MVC framework, like Vue would be useful.

One maybe not-so-elegant JS solution would be to render the content both as source and tabular and have the UI button toggle the visibility property of the appropriate element.

A solution in Vue would likely have configurable template styling based on the view.

[KN4CK3R]

Sure, the "render both" approach works but should not be preferred. The same problem occurs with my second (not pushed) PR for SVG diffs.