View Issue Details

IDProjectCategoryView StatusLast Update
0020351MMW 5Otherpublic2024-05-20 18:34
Reporterzvezdan Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionreopened 
Target Version5.1Fixed in Version5.1 
Summary0020351: Too much empty space on the add-on's installation dialog box
DescriptionThere is too much empty space between "You should only install..." text and the Install Now/Cancel buttons. It appears both when the License box is displayed or not.
TagsNo tags attached.
Attached Files
Fixed in build3022

Activities

Ludek

2023-12-06 21:19

developer   ~0073731

Fixed in 5.1.0.2827

lowlander

2023-12-07 20:12

developer   ~0073747

Verified on 2827.

zvezdan

2023-12-08 08:04

updater   ~0073754

Not perfect. It is ok with addons without license or with the narrowly formatted license text, but still has empty space with the wide license text. The screenshot follows...

Ludek

2023-12-08 14:27

developer   ~0073759

Fixed in 2828

peke

2023-12-16 00:26

developer   ~0073875

Verified 2828

zvezdan

2023-12-18 09:30

updater   ~0073888

It seems to me having the same size. It is really not a big deal, but it would be nice if you can make it better.

Ludek

2023-12-18 13:09

developer   ~0073892

This is the default on the default Material Design skin (which is fine IMO):
image.png (78,223 bytes)   
image.png (78,223 bytes)   

zvezdan

2023-12-18 16:44

updater   ~0073896

This is not fine IMO. The screenshot will follow, with the default Material Design skin, just another color theme.

By the way, I didn't mention that I am talking about default window size, i.e. what you get immediately after dropping the add-on on the player, without any manual resizing.

zvezdan

2023-12-18 16:45

updater   ~0073897

Here it is:

Ludek

2023-12-18 17:02

developer   ~0073899

Last edited: 2023-12-18 17:03

Are you sure that it is the default size of the dialog? Did you delete persistent.json file before the test?
Note that the last size of this resizeable dialog is restored, so you probably resized the initial default?

zvezdan

2023-12-18 18:08

updater   ~0073901

No, I didn't delete persistent.json file.

In my opinion the program should not remember the size of that dialog box, but it should recalculate its size every time when it is opened based on the displayed/contained controls, to fill it properly, at least it should do that for the height.

Currently, if I install one add-on with a really big license file and I enlarge that dialog to see it better, then I close it (either by Install now or Cancel), and then when I want to install another add-on that has a much smaller license file I will get the size of the previously enlarged window, as it was the case on the previous screenshot.

I think the program already has some dialog boxes that are automatically resized on their opening based on their displayed content. You should just implement the same thing with this dialog.

Ludek

2024-05-20 11:22

developer   ~0075461

Hmm, I still cannot replicate with D&D a MMIP onto the player, it is shown in its default size..
Could you please share the testing mmip that you used in your last test?

zvezdan

2024-05-20 13:47

updater   ~0075463

You could find the test addon several posts above:
https://www.ventismedia.com/mantis/view.php?id=20351#c73889

This happens with any addon with the license text. It is not shown in its default size, but the last size of the dialog box is restored, just as you mentioned in your previous note. In my opinion, neither of these two solutions shouldn't be applied. It would be better if the program could detect the length of the license text and automatically resize the dialog box based on that, making its text box large enough to show text without scrollbars, but also without any extra empty space.

I described the steps to reproduce the problem:
- open any addon with the license text and enlarge its dialog box vertically to cover the screen;
- cancel its installation;
- open any addon with the license text -> it will have the previously enlarged size, even if it has smaller (shorter) license text.

Ludek

2024-05-20 18:34

developer   ~0075464

Thanks, fixed in 3022