GraxCode / threadtear

Multifunctional java deobfuscation tool suite
GNU General Public License v3.0
897 stars 123 forks source link

UI improvements. #8

Closed weisJ closed 4 years ago

weisJ commented 4 years ago

Some recommendations for improving the UI.

Some points regarding darklaf

GraxCode commented 4 years ago

Hey weisJ, thanks for your suggestions. I can't seem to find version 2.0.2 on maven. Also DarkBorders#createLineBorder does nothing for me, it displays no border (could be the old version). I can't really fulfill your recommendations without a newer version on maven.

weisJ commented 4 years ago

I can't seem to find version 2.0.2 on maven.

The artifact was renamed to darklaf-core a few versions ago. See https://github.com/weisJ/darklaf/blob/master/README.md#artifacts .

Also DarkBorders#createLineBorder does nothing for me, it displays no border (could be the old version).

It is due to the old version.

GraxCode commented 4 years ago

I have switched to the latest version, and I can see that a few things have changed. But what happened to the separator? http://prntscr.com/s2cupr Also is there a way to disable those vertical lines on the divider?

GraxCode commented 4 years ago

In terms of icons i can't seem to find the icons i need. It's just too big. Do you know the location where they could be?

weisJ commented 4 years ago

I have switched to the latest version, and I can see that a few things have changed. But what happened to the separator? http://prntscr.com/s2cupr Also is there a way to disable those vertical lines on the divider?

The line have been added as most use cases require them to be present. What you can do is replace it with a single line

splitpane.putClientProperty("JSplitPane.style", "line");

or make it invisible

splitpane.putClientProperty("JSplitPane.style", "line");

and add manual spacing between the left and right component,

In terms of icons i can't seem to find the icons i need. It's just too big. Do you know the location where they could be?

They are located here

GraxCode commented 4 years ago

I already browsed that icon path, didn't find them :/ I also noticed that the gap between the left side and the frame icon is smaller, is it possible to revert this change?

weisJ commented 4 years ago

class,enum,folder,field,interface,method,package

I'm not quite sure about the "execution_*" icons. Did you make them yourself? Are they base off any intellij icons (if yes where are they visible).

GraxCode commented 4 years ago

They are based on the schema icon (https://resources.jetbrains.com/help/img/idea/2020.1/database-openapi.icons.schema@2x.png). Thanks for your help!

weisJ commented 4 years ago

https://jetbrains.design/intellij/resources/icons_list/

Search -> "Schema"

weisJ commented 4 years ago

I also noticed that the gap between the left side and the frame icon is smaller

Good catch!

is it possible to revert this change?

Yes :D

GraxCode commented 4 years ago

How? Also you mistakenly put the line splitpane.putClientProperty("JSplitPane.style", "line"); twice, what would be the other case where i hide it? And is the thing with the JSeparator a bug? (https://prnt.sc/s2cupr)

GraxCode commented 4 years ago

It also seems like JTextArea.setMargin is not working anymore, and you should add a border to JDialogs if possible http://prntscr.com/s2hhkc

weisJ commented 4 years ago

How? Also you mistakenly put the line splitpane.putClientProperty("JSplitPane.style", "line"); twice, what would be the other case where i hide it?

It should have been "invisible" the second time. In the next version (should be tomorrow) you'll be able to specify "gripBorderless" which reproduces the old behaviour.

And is the thing with the JSeparator a bug? (https://prnt.sc/s2cupr)

Not really a bug. Separators are really intended to be used in Toolbars etc.. They aren't an replacement for borders. Nonetheless I have adjusted them to paint the whole space they occupy.

It also seems like JTextArea.setMargin is not working anymore

Fixed (with next version)

add a border to JDialogs if possible http://prntscr.com/s2hhkc

Not really possible. What did you do to achieve that the windows doesn't have a shadow?

For comparison how it should look like: dialog (No border really necessary here)

weisJ commented 4 years ago

If you really want to have the border you can use LafManager#registerInitTask in which you set the "RootPane.dialogBorder" property.

GraxCode commented 4 years ago

I think the shadow looks better, but why is it not displaying?