Here is an updated patch. Also here is a link for everyone to see what we could make Java tab look like: https://lh5.googleusercontent.com/-lzTcvXoWhOw/UBq_VGFqL2I/AAAAAAAAACc/TnxVpfV5WL0/s937/Advanced_Options.jpg Notes: 1. Changed name to "Advanced" 2. Shrunk Java dialog box 3. Added bottom section for experimental features This would clear up enough space in the general tab for me to move the measurement units over. I've cc'ed the ux team since this is now a visual thing also. Please respond to the dev mailing list as I'll be pushing out a few patches corresponding to these changes. Joel On Thu, Aug 2, 2012 at 11:29 AM, Stefan Knorr <heinzlesspam@gmail.com>wrote:
Hello there, On Thu, 2012-08-02 at 08:14 -0700, Joel Madero wrote:Perfect idea! I'm going to go ahead and work on that. I will take out "limited" (I have no clue what it means either, I figured someone else must have known when they added it ;)).Well, hopefully, I didn't sound overly assertive here.I have a quick question though. What's the best way to modify a patch? The only way I know how is to go back to dismiss my changes and start from scratch. I'm sure there is a better way that I don't know of.So, I am by no means the git expert here (I still barely understand the basics of it) but ... you probably still have the original patch committed locally, (and if not, you can do $ git am path/to.patch ), so, do all the modifications you want to do, then do the usual $ git add . $ git commit At the end, you can just use $ git rebase -i HEAD~2 to either squash (merge the commits and edit the commit message) or fixup (just merge the commits) (assuming you had only your original commit and the new one = HEAD~2). Also: Note that you should only ever use rebase -i on local commits – modifying the global commit history is evil. (It won't do any harm to everyone else, if you don't have commit access, but still.) Modifying the patch in a text editor can easily go wrong, so that's not recommended. Astron.
Attachment:
0001-fdo-52636-Added-Sectional-Line-and-Header-for-Experi.patch
Description: Binary data
