Skip to content
This repository has been archived by the owner on Jul 10, 2023. It is now read-only.

Added basic file explorer functionality #1894

Open
wants to merge 2 commits into
base: alpha
Choose a base branch
from

Conversation

b10z
Copy link
Contributor

@b10z b10z commented Apr 13, 2020

Hey guys! Here i have some basic file explorer functionality (Delete file, rename file and bar on the top which allows to browse on specific folder on our content path. I attach a gif with the new interface adjustments.
20200414_063045
(edit: new gif!)
The code hasn't been fully tested on windows, since i could not make my keyboard to work correctly at any time with reicast for some reason. Some of the buttons that worked tho, were Crtl+V so that was enough for input checks, examples, etc but still not sure if everything works.
Also, i tried to keep my code simple and not that hacky and i think that its ok most of the time. Any suggestion/opion is welcome ofc!

Also, im almost done with archives. Sorry for the whole delay, but i realized that i needed threads to work a little too late and i had to write my code from scratch.

Edit: Everything i mention bellow i think its done plus some changes to what i had done. Added some comments as well. :) No bugs or errors for me on android 9, but havent test rename function on windows yet.

@b10z
Copy link
Contributor Author

b10z commented Apr 13, 2020

Thinking about adding a new scrollable which includes all the files from multiple paths at once. Currently this code supports multiple paths but each path shows up only its own files and directories. (So you cant view content of /ReicastHome1/ and /ReicastHome2/ at once -you can do this separately from the bar- only if the path is like this/ReicastHome1/ReicastHome2). Also i believe i need to add a slash or something to each subfolder just to distinguish them from content paths used.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant