Attaching a proposed patch that will add information about fbrun in the fluxbox-config.xml guide.
Created attachment 58351 [details, diff] fluxbox-config_fbrun.patch well, here it is. smithj, please take a look and, if everyone is ok with this, gimme a green light to commit it.
If you wonder, I have intentionally omitted the date and version tags, as I consider that this is the commiter's job, not the patch-proposer's one.
i don't have to give you the green. you are the dev, not me ;-) however, there are a few things i would change, for instance, the location of this section. gimme a day or so and i will think about it (assuming you want my input)
I put the section there, because the next ones (system monitor, desktop icons, etc) were based on external applications and this one was a fluxbox internal. Anyway, I'll let you think about the section location.
Alrighty, here is the patch to current CVS doc and a separate file which contain only the 'Fluxbox menu' section (without the + chars at the beginning of the line, added by diff). Use either of them. Of course, any feedback appreciated.
Created attachment 58509 [details, diff] fluxbox-config_menu.patch here's the patch.
Created attachment 58511 [details] fluxbox-config_menu.txt and here's the section, if you want to add it in another place in the doc.
Created attachment 58533 [details, diff] fluxbox-config_megapatch.xml.diff consider all the other patches obsolete by this one. i can't mark them that way, but all of the previous patches are slightly modified and included in this diff. alin, i changed the wording in a few places not so much because it was wrong, but to make it consistant with the rest of the guide. other than that, very nice patches :-) thank you very much version bump from 1.0.4 to 1.1.0 due to relatively large revisions added alin as an editor due to his work on this doc and his discussion with me about it in irc. this patch includes editing the menu file by hand, file managers in flux, picture viewers in flux, fbrun, and menu/key shortcuts for the above.
oh, you can also view the rendered html of the new doc at http://neverland.ncssm.edu/~smithj/gentoo/fluxbox-config.html
I have marked all the other attachments as obsolete. Swift, can you take a look at the patch and give me a green light to commit it (or commit it yourself, if ok)?
I took a look at the online version, guess we can commit these changes. Good work Jonathan & Alin :)
alin, i think we are ready to commit. one thing, however. in your section (on fbrun), replace RUN with Run... per request of some folks in #gentoo-doc i could create a patch, but i'm not sure that its needed. just apply the megapatch and replace that if you think its necessary. good work all
this evening, after i'll get home, i'll apply the patch and s/RUN/Run/g, then xmllint --valid --noout fluxbox-config.xml && cvs ci ;-)
Created attachment 58564 [details, diff] fluxbox-config_megapatch.xml.diff this fixes a few tpyos and the RUN thingy. just apply it and commit
Created attachment 58574 [details] fluxbox-config_megapatch.xml.diff.patch Please consider this patch to the patch (!). There should be one hunk failed when you patch, just ignore.
Checked and commited. Thank you all!
