You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@netbeans.apache.org by GitBox <gi...@apache.org> on 2023/01/11 10:26:39 UTC

[GitHub] [netbeans] mbien opened a new pull request, #5274: Gradle options UI cleanup (Execution and Appearance panel).

mbien opened a new pull request, #5274:
URL: https://github.com/apache/netbeans/pull/5274

    - consistent gaps and component anchors
    - moved a few options to the global exec group
   
   please note that the before screenshots are actually NB 16 and not master. So they have 1-2 fields less.
   
   before (checkboxes in upper panel are all over the place + panel gaps + some exec options not in the lower panel):
   ![gradle-before](https://user-images.githubusercontent.com/114367/211778476-3d357b70-61e7-423f-970d-d8929098bff8.png)
   
   after:
   ![gradle-after](https://user-images.githubusercontent.com/114367/211778489-012d686c-c77a-47f6-bcf8-3920968434d9.png)
   
   before (left gaps on checkboxes):
   ![gradle-app-before](https://user-images.githubusercontent.com/114367/211778329-69ce359b-1b05-498a-aeab-610aed62199b.png)
   
   after:
   ![gradle-app-after](https://user-images.githubusercontent.com/114367/211778428-02506a6d-4dfb-4636-8231-9e54840331b0.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


[GitHub] [netbeans] mbien commented on pull request #5274: Gradle options UI cleanup (Execution and Appearance panel).

Posted by GitBox <gi...@apache.org>.
mbien commented on PR #5274:
URL: https://github.com/apache/netbeans/pull/5274#issuecomment-1379366976

   I only wanted to fix some layout issues in the UI, consistent gaps, resizing etc, Moving things around to entirely different option panels would be more than just layout adjustments and something for another PR.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


[GitHub] [netbeans] mbien commented on pull request #5274: Gradle options UI cleanup (Execution and Appearance panel).

Posted by GitBox <gi...@apache.org>.
mbien commented on PR #5274:
URL: https://github.com/apache/netbeans/pull/5274#issuecomment-1379999400

   @lkishalmi updated the `after` image on the PR with the latest change.
    - added Maven card as suggested and moved the "Prefer Maven" checkbox over
    - kept the left aligned checkboxes of the screenshot above
    - some more tweaking for gap consistency between all cards


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


[GitHub] [netbeans] lkishalmi commented on pull request #5274: Gradle options UI cleanup (Execution and Appearance panel).

Posted by GitBox <gi...@apache.org>.
lkishalmi commented on PR #5274:
URL: https://github.com/apache/netbeans/pull/5274#issuecomment-1379347668

   Well, I actually like the first new version. The second one looks too much for me.
   I'd offer a third option then. Let's create a `Maven` specific panel and put that option there.
   
   There might be other new options in that area...
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


[GitHub] [netbeans] swpalmer commented on pull request #5274: Gradle options UI cleanup (Execution and Appearance panel).

Posted by GitBox <gi...@apache.org>.
swpalmer commented on PR #5274:
URL: https://github.com/apache/netbeans/pull/5274#issuecomment-1379361487

   The prefer Maven over Gradle option really isn't an option of Maven or Gradle.  It needs to be in a more general location, possibly on the general tab, even though (currently) these are Java-related.  Theoretically Gradle projects can support C++ as well so if that was ever pursued things could get messier still.
   Also the tab order is a little odd, Ant, Gradle, and Maven are related build systems, they should be grouped together in the tab order.  I'm not sure if that is easy to do or if the order is somewhat ad-hoc because everything is a plugin or something like that.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


[GitHub] [netbeans] mbien commented on pull request #5274: Gradle options UI cleanup (Execution and Appearance panel).

Posted by GitBox <gi...@apache.org>.
mbien commented on PR #5274:
URL: https://github.com/apache/netbeans/pull/5274#issuecomment-1379350402

   another proposal (all checkboxes aligned in left column):
   ![gradle-after4](https://user-images.githubusercontent.com/114367/211894747-984dcdc5-78e4-4a0c-b612-3660a349480b.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


[GitHub] [netbeans] mbien merged pull request #5274: Gradle options UI cleanup (Execution and Appearance panel).

Posted by GitBox <gi...@apache.org>.
mbien merged PR #5274:
URL: https://github.com/apache/netbeans/pull/5274


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


[GitHub] [netbeans] mbien commented on pull request #5274: Gradle options UI cleanup (Execution and Appearance panel).

Posted by GitBox <gi...@apache.org>.
mbien commented on PR #5274:
URL: https://github.com/apache/netbeans/pull/5274#issuecomment-1379326713

   @lkishalmi tbh I don't quite like how this looks. Can I wrap this into an "Other Options" panel?
   ![gradle-after2](https://user-images.githubusercontent.com/114367/211889078-14d8a9b8-a509-494b-af1e-afb8d4750835.png)
   
   other options panel:
   ![gradle-after3](https://user-images.githubusercontent.com/114367/211890674-92a144fd-8789-49a3-b892-483da4a05bda.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


[GitHub] [netbeans] lkishalmi commented on pull request #5274: Gradle options UI cleanup (Execution and Appearance panel).

Posted by GitBox <gi...@apache.org>.
lkishalmi commented on PR #5274:
URL: https://github.com/apache/netbeans/pull/5274#issuecomment-1379605963

   @swpalmer That would be a major re-arch. We should create a Build Tools section in the Options Dialog.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists