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

open text files in filemanager #8728

Closed
wants to merge 1 commit into from
Closed

open text files in filemanager #8728

wants to merge 1 commit into from

Conversation

ronie
Copy link
Member

@ronie ronie commented Dec 31, 2015

this adds the option to display the content of textfiles in the filemanager.

txt

@ronie ronie added Type: Feature non-breaking change which adds functionality Component: GUI engine labels Dec 31, 2015
@BigNoid
Copy link
Member

BigNoid commented Dec 31, 2015

Nice one. From code it looks like you want xsp and m3u files to open with the textviewer too? That doesn't work on my end, it opens those as it were directories.

@ronie
Copy link
Member Author

ronie commented Dec 31, 2015

ah yeah, silly me. i'll remove those.
playlists should be opened by the player.

@msduketown
Copy link

Why is that close-button on the left?

@ronie
Copy link
Member Author

ronie commented Dec 31, 2015

usually a close button will close the dialog...
i have no reason to believe it acts differently in this case.

@msduketown
Copy link

All other dialogs have a close-button on the right...

edit: And I think centring (like #8242) would look so much better. Maybe even a candidate to reduce dialogs.xml's further.

@ronie
Copy link
Member Author

ronie commented Dec 31, 2015

right

right2

@MrMC
Copy link

MrMC commented Dec 31, 2015

might want to sanity check the file size or someone is going to try and open a huge log file :)

@guidosarducci
Copy link

This is wonderful to have and feels long overdue. Pending defensive/sanity changes as noted above, my only question is if this could make it into Jarvis too?

@razzeee
Copy link
Member

razzeee commented Jan 1, 2016

Nope, Jarvis is feature frozen.

@ronie ronie added the WIP PR that is still being worked on label Jan 3, 2016
@ronie
Copy link
Member Author

ronie commented Feb 1, 2016

updated. limited the buffer length, as suggested.

@ronie
Copy link
Member Author

ronie commented Feb 5, 2016

jenkins build this please

@Paxxi
Copy link
Member

Paxxi commented Feb 5, 2016

There's still no sanity check on the size and there should probably be some error handling and not just display an empty window. I would defer window loading and setting the header until after the file is sanity checked. This will fail on win32 log file I guess as it's using utf16 for some reason and this assumes encoding is ascii or utf8

@stefansaraev
Copy link
Contributor

ready to go ?

@ronie
Copy link
Member Author

ronie commented Apr 11, 2016

judging by the comments, i'd say it's far from ready to go.
i'm afraid i have no idea how to fix those issues.

@ronie
Copy link
Member Author

ronie commented Nov 24, 2017

closing, superseded by #13093

@ronie ronie closed this Nov 24, 2017
@ronie ronie deleted the textviewer branch May 7, 2019 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: GUI engine Type: Feature non-breaking change which adds functionality WIP PR that is still being worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants