- Notifications
You must be signed in to change notification settings - Fork914
Open Favorites tab on first project or file open#8908
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
7492b68 toa229550Compareplatform/favorites/src/org/netbeans/modules/favorites/Module.java OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
eirikbakke commentedOct 13, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
platform/favorites/src/org/netbeans/modules/favorites/Module.java OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
neilcsmith-net commentedOct 13, 2025
I think this should probably be open by default unless ergonomics is in play, and the logic for opening with files handled in there? |
mbien commentedOct 13, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
not completely sure what open by default means. If you mean to open it on first launch without any trigger, I thought about it and tried it, but it looks weird next to the welcome page. Your comment at#8744 (comment) sounded like you don't like that idea either so I didn't even mention it in the dev list discussion thread. |
neilcsmith-net commentedOct 13, 2025
Sorry, a wrong assumption on how this was handled based on long time platform projects that have obviously overridden the defaults. If it behaves similarly to Project and Files, both with or without ergonomics, all good! |
mbien commentedOct 13, 2025
oh wow. I was actually searching for that. For some reason I didn't make the connection that the west column is called navigator. I assumed thatwindowmanager.wswmr would set the initial layout, but editing numbers there did nothing. I wanted to reduce that value to give the editor more space. Its unfortunate that it only supports relative values for initial sizing. 25% would still work on 720p according to my measuring stick, assuming the window is maximized first. Should we change it? btw the optimal value for 1080p would be ~16% when maximized,half of the current width ;) |
a229550 toc178133Comparembien commentedOct 14, 2025
made two changes:
|
6ed0a4d to43824f0Compareeirikbakke commentedOct 14, 2025
Do MacOS users normally maximize their windows? I had to install a special window tiler app (SizeUp) to make maximization easy... otherwise if you press the green window button thing it just goes into a very useless full screen mode thing. In my platform app I have special code for maximizing the window on startup. But the "out of the box" experience for the NetBeans IDE might well be the narrower unmaximized state. |
mbien commentedOct 14, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I don't know, but I had raspi users and embedded in mind for the 720p scenario. For mac even the windowed mode will be larger than 720p. On 1920x1080 NB opens as ~1700pix wide window where 25% is already wider than necessary. |
neilcsmith-net commentedOct 14, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I'm a little sceptical at adding a taband lowering the width in the same PR if you're targetting freeze. At least one system I tested on the mailing list thread was fairly tight with all four. UI scaling also has an impact. OTOH, the width can always be put back in at a later RC I guess. 😄 EDIT: Mind you, has this changed? Does this not also depend onhttps://github.com/apache/netbeans/blob/master/platform/core.ui/src/org/netbeans/core/ui/resources/windowmanager/explorer.wsmode (platform) andhttps://github.com/apache/netbeans/blob/master/ide/projectui/src/org/netbeans/modules/project/ui/resources/explorer-override.wsmode (IDE) ? |
eirikbakke commentedOct 14, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Agree with this. For making the sidebar narrower, I think it would be better to add a feature to the window system to allow a constant pixel width to be specified as the initial default. (EDIT: Exactly how to do this might need some more thought. What happens if a constant width is specified initially and then the window is then resized etc.) |
mbien commentedOct 14, 2025
going to revert the 5% reduction. Please be specific while reviewing or discussing. I heard that several times that the sidebar doesn't fit the tabs, but couldn't reproduce it even once. In all realistic cases it was too large or far too large (like the 1080p case). Matthias did try to reproduce the issue too during the discussion on the dev list since it was mentioned there too and he couldn't either. Lets not waste all our time by being so vague. |
- tab group will now contain 4 views: Projects, Files, Favorites and Services - will be opened with the already existing on-first-project-open mechanism - the very first on-editor-open event will also open the Favorites tab
43824f0 to852a308Compareeirikbakke commentedOct 14, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
eirikbakke left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Tested it on a clean NetBeans install (deleted userdir) and it works fine. Looked over the mailing list discussion, most seem in favor, main issue is minimum width of navigator, which is still OK as of this PR. (Could be decreased in the future on wider screens if minimum width could be stated in pixels rather than %age of screen size.)
mbien commentedOct 14, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
thanks for testing! The long term solution would be likely setting fixed initial sizes in px (as mentioned before), but they would have to scale with the UI scaling factor. 25% on 1080p looks nearly half empty. So it makes sense that when scaled 2x it would be slightly too small (not accounting for different fonts and other factors which might ask for more wiggle room). (this PR reverted to 30% again, making no changes to the layout) |
eirikbakke commentedOct 14, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
mbien commentedOct 14, 2025
right. That is what I mean since swing pixels do scale already. The west column needs about 350 pixels right now (includes extra wiggle room for fonts) when scaling is disabled. For 200% scaling it would need about twice as much, but the scaling factor itself would do that already. |
41def67 intoapache:masterUh oh!
There was an error while loading.Please reload this page.
mbien commentedOct 14, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
a platform test is failing. will investigate. see#8915 |
neilcsmith-net commentedOct 15, 2025
Thanks for doing this! Just because I've only just had a chance to follow up ...
Me neither, although the machine I'm using now while travelling is tightest. It's an Arm Chromebook - native resolution is actually 1920x1200, but the default (and good for my eyes!) is scaled to 1440x900. That can actually go to 24% before the tabs don't fit. That does assume no project group is set though, and this does cover reset windows as well as first run. If you come back to this then the two |




Uh oh!
There was an error while loading.Please reload this page.
made west column 5% smallerscreenshottoo small with 200% scaling