Davilarek / Vencord

The cutest Discord client mod + BD plugin support
GNU General Public License v3.0
18 stars 2 forks source link

feat(plugin): Update FileSystemViewer UI top-row button redesign #19

Closed net-wayfarer closed 2 months ago

net-wayfarer commented 2 months ago

Description of Changes

Update FileSystemViewer UI top-row button redesign to better suit the layout Vencord has made.

Before: before

After: after

Rationale behind Changes

Vencord updated the button layout for the buttons atop both the Settings tab as well as Themes tab. This change is heavily inspired by the updates made in Vencord, and has the potential to allow custom icons to be used alongside the buttons in the redesign.

Suggested Testing Steps

Davilarek commented 2 months ago

Here's how it looks on my display: obraz In my opinion, this looks weird being outside of the main box: obraz

@zrodevkaan what you think about this redesign?

net-wayfarer commented 2 months ago

I'm not entirely sure on how to have the Size of /: entry in the same area as those buttons.

zrodevkaan commented 2 months ago

Here's how it looks on my display: obraz In my opinion, this looks weird being outside of the main box: obraz

@zrodevkaan what you think about this redesign?

My apologizes. I did not see this till now.

Very nice UI. Although please do work on the actual FS SystemView if you can. Its just slashes. Id like to see if you could change that up a bit ?

Davilarek commented 2 months ago

wdym "just slashes"? https://en.wikipedia.org/wiki/Filesystem_Hierarchy_Standard?lang=en#:~:text=in%20the%20fhs%2C%20all%20files%20and%20directories%20appear%20under%20the%20root%20directory%20%2F%2C

In the FHS, all files and directories appear under the root directory /

zrodevkaan commented 2 months ago

wdym "just slashes"? https://en.wikipedia.org/wiki/Filesystem_Hierarchy_Standard?lang=en#:~:text=in%20the%20fhs%2C%20all%20files%20and%20directories%20appear%20under%20the%20root%20directory%20%2F%2C

In the FHS, all files and directories appear under the root directory /

/ is not what I meant directly. The UI just needs to be changed instead of white bold stokes around the border.

Davilarek commented 2 months ago

I feel that's out of scope of this PR. "feat(plugin): Update FileSystemViewer UI top-row button redesign"

zrodevkaan commented 2 months ago

Thats the point of a question and asking. Not making.

Its possible said person could.

Davilarek commented 2 months ago

obraz I managed to place it here, does it look okay or nay?

Davilarek commented 2 months ago

obraz Or this @net-wayfarer @zrodevkaan

zrodevkaan commented 2 months ago

obraz Or this @net-wayfarer @zrodevkaan

Size of looks abnormal.

Davilarek commented 2 months ago

What?

zrodevkaan commented 2 months ago

It looks weird.. ?

Davilarek commented 2 months ago

why?

net-wayfarer commented 2 months ago

obraz Or this @net-wayfarer @zrodevkaan

I'd say that just creates ambiguity, end users may/may not know if that Size of: is actually a button or are the rest of the buttons are purely cosmetic (has no functionality). If it was coloured or shaped differently to the rest of the buttons, it can prevent that unwanted confusion. Then again that's not my call.

Davilarek commented 2 months ago

obraz I managed to place it here, does it look okay or nay?

Okay, so is this one better?

net-wayfarer commented 2 months ago

mockup

Or you could have it like this, inspired by GitHub looks for navigation, also breadcrumb navigation.

This is only a mockup, there's no code for this.

Update: Directories should be at the top, perhaps with their own icon to set it apart from regular files within the tree. Also there should be a way to navigate back to parent directory, but that's also missing.

Davilarek commented 2 months ago

I'll consider making something like this but not right now, let's focus on this PR. I think the current version is enough and satisfies me enough. Marking as ready to merge.

Davilarek commented 2 months ago

This will be squashed on merge. @zrodevkaan do you approve these changes?