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

Line wrapping in the newest version #6064

Closed
2 of 7 tasks
theAkito opened this issue Feb 13, 2019 · 10 comments · Fixed by #6074
Closed
2 of 7 tasks

Line wrapping in the newest version #6064

theAkito opened this issue Feb 13, 2019 · 10 comments · Fixed by #6074
Labels
Milestone

Comments

@theAkito
Copy link

  • Gitea version (or commit ref): 7a72aa6
  • Git version: 2.11.0
  • Operating system: DietPi v6.21.1
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes
    • No
    • Not relevant
  • Log gist:

Description

I just updated from 0b51072 to 7a72aa6 . I noticed that files now always include line breaking/wrapping and I didn't find a way to turn it off. I assume this is a new "feature", as this is also confirmed on try.gitea.io.

Another issue is, that in a file I have the line numberings suddenly stop at a certain point. It just happens in this specific file and I can't reproduce this on try.gitea.io with random input. Didn't try the original input, as the file has confidential data in it.

Screenshots

2019-02-14 00_47_17-gitea-error1

@oscarlofwenhamn
Copy link
Contributor

Do you have a link to the example on try.gitea.io, to illustrate the issue?

@lafriks
Copy link
Member

lafriks commented Feb 14, 2019

Example: https://try.gitea.io/Etzelia/plural/src/branch/master/plural-gen/ordinals.xml.go

@lafriks lafriks added this to the 1.8.0 milestone Feb 14, 2019
@oscarlofwenhamn
Copy link
Contributor

It also looks like this is suffering from the same issue with highlighting as I brought up here #6057.

@theAkito
Copy link
Author

@lafriks @oscarlofwenhamn Thank you for the attention to this bug. I hope this can be resolved effectively.

@silverwind
Copy link
Member

silverwind commented Feb 14, 2019

This is related to wrapping introduced in #5927. We need to change how line numbers are generated, ideally using CSS counters and ::before. I will have a look a bit later.

@theAkito
Copy link
Author

theAkito commented Feb 14, 2019

This is related to wrapping introduced in #5927. We need to change how line numbers are generated, ideally using CSS counters and ::before. I will have a look a bit later.

Well, so this mess is actually a feature... I need this solved ASAP, or I have to stay on a previous version of Gitea, because this is a really terrible mess, I can't really focus on doing my work due to this.

Or I have to remove e5cd80e manually...

@silverwind
Copy link
Member

I'm happy if collaborators agree to reverting #5927 as I'm not sure when I get around to a proper fix which will take some time.

silverwind added a commit to silverwind/gitea that referenced this issue Feb 14, 2019
techknowlogick pushed a commit that referenced this issue Feb 15, 2019
@theAkito
Copy link
Author

Thanks for a quick resolution. 👍

@oscarlofwenhamn
Copy link
Contributor

FTR: The highlighting still looks to be broken even after this PR.

@silverwind
Copy link
Member

#6057 is a different issue, certainly not related here.

@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants