neilcsmith-net commented on PR #5335:
URL: https://github.com/apache/netbeans/pull/5335#issuecomment-1400074529

   That Thunderbird mockup looks worse than the original in my opinion.  I 
think I agree with @mbien here that vertically centring the icon is not the 
answer - fixing sizing probably is.
   
   I don't think this is a good fit for the release given ongoing discussion.  
Putting this back to master with do-not-merge on pending resolution of 
discussion and testing across LAFs.
   
   > I think rebasing is probably not even needed since the branches didn't 
diverge yet.
   
   As clarification on that point, master diverges immediately on branch.  Any 
PR that picked up the spec version change from master would require rebasing.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

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

Reply via email to