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

Resource Properties Dialog to small to see buttons #1895

Closed
jukzi opened this issue May 22, 2024 · 4 comments · Fixed by #1929
Closed

Resource Properties Dialog to small to see buttons #1895

jukzi opened this issue May 22, 2024 · 4 comments · Fixed by #1929
Labels
bug Something isn't working

Comments

@jukzi
Copy link
Contributor

jukzi commented May 22, 2024

  1. package explorer
  2. context menu
  3. Properties
    =>
    image
    image
eclipse.buildId=4.32.0.I20240516-1800
java.version=21.0.2

I don't remember that it used to happen in older eclipse versions (or at least not that much too small), so it might be a regression.

@jukzi jukzi added the bug Something isn't working label May 22, 2024
@jukzi
Copy link
Contributor Author

jukzi commented May 22, 2024

That page is org.eclipse.ui.internal.ide.dialogs.ResourceInfoPage, but i could not find a related commit to that class causing the issue. May be some general change?

@HeikoKlare
Copy link
Contributor

The last update of that page I am aware of was in December: #1300

But from a first glance, I do not see that this change affected the general layout of that page. If I am not mistaken, I see the same behavior in an Eclipse 2020-03 IDE:

image

In my opinion, the problem is that the text field(s) (in particular the one for the location) are sized according to their contents instead of according to the size of the container. I (personally) would prefer to scroll within the location text box rather than scrolling the whole page.

@merks
Copy link
Contributor

merks commented May 22, 2024

Yes, I think it's been this way for a long time and one just doesn't notice until the location is grossly long.

One thing that has changed is this dialog remembers its bounds:

image

So is now much more likely to come up narrow where the long location doesn't fix.

@jukzi
Copy link
Contributor Author

jukzi commented Jun 5, 2024

This is a combination of two bugs:

  1. the "Location" text is (unlike the "Path" text) not wrapped into multiple lines
  2. regression from Store properties dialog bounds. #1282 @marcushoepfner that the preferred size of the page is not used anymore: The PreferenceDialog has a very custom size algorithm in org.eclipse.jface.preference.PreferenceDialog.showPage(IPreferenceNode) that tries to adapt the size of the dialog to the preferred size of the current page. That does not work anymore as for the first page always just the old size is used. Now when switching between different preference pages the size still grows a bit but not enough.

jukzi pushed a commit to jukzi/eclipse.platform.ui that referenced this issue Jun 5, 2024
jukzi pushed a commit to jukzi/eclipse.platform.ui that referenced this issue Jun 5, 2024
…eclipse-platform#1895

This reverts commit 8abe1b1.

The PreferenceDialog has a custom size calculation that tries to adapt
to the current page. That contradicts using the the former size

eclipse-platform#1895
@jukzi jukzi linked a pull request Jun 5, 2024 that will close this issue
jukzi pushed a commit to jukzi/eclipse.platform.ui that referenced this issue Jun 5, 2024
…eclipse-platform#1895

This reverts commit 8abe1b1.

The PreferenceDialog has a custom size calculation that tries to adapt
to the current page. That contradicts using the the former size

eclipse-platform#1895
jukzi pushed a commit to jukzi/eclipse.platform.ui that referenced this issue Jun 5, 2024
jukzi pushed a commit to jukzi/eclipse.platform.ui that referenced this issue Jun 5, 2024
…eclipse-platform#1895

This reverts commit 8abe1b1.

The PreferenceDialog has a custom size calculation that tries to adapt
to the current page. That contradicts using the the former size

eclipse-platform#1895
jukzi pushed a commit to jukzi/eclipse.platform.ui that referenced this issue Jun 6, 2024
jukzi pushed a commit to jukzi/eclipse.platform.ui that referenced this issue Jun 6, 2024
…eclipse-platform#1895

This reverts commit 8abe1b1.

The PreferenceDialog has a custom size calculation that tries to adapt
to the current page. That contradicts using the the former size

eclipse-platform#1895
jukzi pushed a commit that referenced this issue Jun 6, 2024
jukzi pushed a commit that referenced this issue Jun 6, 2024
This reverts commit 8abe1b1.

The PreferenceDialog has a custom size calculation that tries to adapt
to the current page. That contradicts using the the former size

#1895
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants