Angband.oook.cz
Angband.oook.cz
AboutVariantsLadderForumCompetitionComicScreenshotsFunniesLinks

Go Back   Angband Forums > Angband > Development

Reply
 
Thread Tools Display Modes
Old August 29, 2011, 21:11   #1
fph
Knight
 
Join Date: Apr 2009
Location: Berlin / Italy
Posts: 776
fph is on a distinguished road
RST help files

(well, this time I decided to open a new thread instead of keep on going OT on this old one)

There is still some minor work to be done, but I think I have an usable version of the RST online help. Apart from the converted help files, there is:
-"make manual.html" and "make manual.pdf" that create HTML and PDF version
-the online help viewer skips "RST directives" (only when they are one-liners, for now, though)
-some modifications to files.c to parse menu and link targets directly from (a very limited subset of) RST.

What still needs to be done:
1. beautify the output: a better CSS and LaTeX preamble, maybe a couple of images.
2. instruct the online help viewer to render a couple of RST constructs in a more readable version (namely, |something| and "\ ", "\\").
3. skip RST directives as whole paragraphs.
4. there's a couple of warnings, that I think are due to outdated information in the docs (do we still have view_special_light and view_granite_light?). I could easily remove them, but I'd like someone to confirm me that they can go.
5. how can I get "make clean" to work properly, without mkmk complaining? The previous help files managed, but I can't understand what did the trick in the Makefile.
6. pressing ? at the right moment during character creation should lead directly to the classes and races description, but this doesn't work in the master branch either -- can you confirm that?
7. at some point, a weird bug in my editor removed some lower-case "s" letters at word start (e.g.: similar -> imilar). I think I have now fixed all the damage, but if you're skimming through the help files please keep your eyes open for words (not) beginning with "s".

The help file branch is online at https://github.com/fph/angband. After we fix items 4 and 5 (I need a quick consult for that), probably I can issue a pull request. Items 1, 2, 3 and 6 can be a later goal (there is no obvious regression).
__________________
Dive fast, die young, leave a high-CHA corpse.
--
You read a scroll labeled 'lol gtfo' of Teleport Level.
fph is offline   Reply With Quote
Old August 29, 2011, 21:16   #2
fph
Knight
 
Join Date: Apr 2009
Location: Berlin / Italy
Posts: 776
fph is on a distinguished road
Oh, and about the *slay* problem: I think I have solved it using RST substitution directives: currently, the string "*slay*" is displayed as "|*slay*|", with an extra pair of |...|, in the help files, and correctly in HTML/PDF. If I/we implement item 2 in my list (which doesn't sound complicated), we can get rid of the |...| as well. Probably the cleanest way is converting them to spaces in the online help viewer.
__________________
Dive fast, die young, leave a high-CHA corpse.
--
You read a scroll labeled 'lol gtfo' of Teleport Level.
fph is offline   Reply With Quote
Old August 29, 2011, 23:05   #3
d_m
Angband Devteam member
 
d_m's Avatar
 
Join Date: Aug 2008
Location: Philadelphia, PA, USA
Age: 39
Posts: 1,516
d_m is on a distinguished road
Quote:
Originally Posted by fph View Post
(well, this time I decided to open a new thread instead of keep on going OT on this old one)

There is still some minor work to be done, but I think I have an usable version of the RST online help.
Awesome! Later tonight I will take a look at this.

I can confirm that in-game help during character creation is borked. I'm not sure if there's a ticket open yet, but there should be. Definitely worth fixing (and probably backporting to 3.3.1 if it's broken in 3.3.0 too).

Thanks for your hard work!
__________________
linux->xterm->screen->pmacs
d_m is offline   Reply With Quote
Old August 30, 2011, 10:11   #4
Magnate
Angband Devteam member
 
Join Date: May 2007
Location: London, UK
Posts: 5,057
Magnate is on a distinguished road
Send a message via MSN to Magnate Send a message via Yahoo to Magnate
Quote:
Originally Posted by d_m View Post
Awesome! Later tonight I will take a look at this.

I can confirm that in-game help during character creation is borked. I'm not sure if there's a ticket open yet, but there should be. Definitely worth fixing (and probably backporting to 3.3.1 if it's broken in 3.3.0 too).

Thanks for your hard work!
+1 that's fantastic work - many thanks.
__________________
"3.4 is much better than 3.1, 3.2 or 3.3. It still is easier than 3.0.9, but it is more convenient to play without being ridiculously easy, so it is my new favorite of the versions." - Timo Pietila
Magnate is offline   Reply With Quote
Old August 30, 2011, 11:04   #5
Nomad
Knight
 
Nomad's Avatar
 
Join Date: Sep 2010
Location: England
Posts: 958
Nomad is on a distinguished road
Quote:
Originally Posted by d_m View Post
I can confirm that in-game help during character creation is borked. I'm not sure if there's a ticket open yet, but there should be. Definitely worth fixing (and probably backporting to 3.3.1 if it's broken in 3.3.0 too).
It's been borked for a long time. I'm pretty sure it stopped working somewhere between 3.0.9 and 3.1.2v2.
Nomad is offline   Reply With Quote
Old August 30, 2011, 21:20   #6
fph
Knight
 
Join Date: Apr 2009
Location: Berlin / Italy
Posts: 776
fph is on a distinguished road
Quote:
Originally Posted by Magnate View Post
+1 that's fantastic work - many thanks.
No need at all -- it's nothing compared to the work you guys do regularly to maintain Angband and develop new features.

Going on to fix the last rough spots...
__________________
Dive fast, die young, leave a high-CHA corpse.
--
You read a scroll labeled 'lol gtfo' of Teleport Level.
fph is offline   Reply With Quote
Old September 4, 2011, 13:34   #7
fph
Knight
 
Join Date: Apr 2009
Location: Berlin / Italy
Posts: 776
fph is on a distinguished road
I've fixed in my list 3, 4, 5 and part of 2 (handling backslashes is kinda tricky). I think this can now safely be pulled into the main development branch (of course after you review the code -- there's some C involved, and it could be buggy or not following your code style).

How is the protocol for that?
__________________
Dive fast, die young, leave a high-CHA corpse.
--
You read a scroll labeled 'lol gtfo' of Teleport Level.
fph is offline   Reply With Quote
Old September 4, 2011, 17:05   #8
d_m
Angband Devteam member
 
d_m's Avatar
 
Join Date: Aug 2008
Location: Philadelphia, PA, USA
Age: 39
Posts: 1,516
d_m is on a distinguished road
Quote:
Originally Posted by fph View Post
I've fixed in my list 3, 4, 5 and part of 2 (handling backslashes is kinda tricky). I think this can now safely be pulled into the main development branch (of course after you review the code -- there's some C involved, and it could be buggy or not following your code style).

How is the protocol for that?
The best way to do it is to open a pull request on Github. You should be able to find instructions on how to do it, but the basic gist is:

1. make your changes in a branch
2. commit your changes
3. push your branch up to your github account
4. open a pull request against the staging branch of angband/angband

That's it! We get notifications whenever someone does this, so then we can go review things, comment on the pull request and/or merge it.
__________________
linux->xterm->screen->pmacs
d_m is offline   Reply With Quote
Old September 5, 2011, 10:45   #9
Magnate
Angband Devteam member
 
Join Date: May 2007
Location: London, UK
Posts: 5,057
Magnate is on a distinguished road
Send a message via MSN to Magnate Send a message via Yahoo to Magnate
fph - thanks again for your work on this.

...

You don't by any chance fancy converting the spoilers do you? They haven't been updated in 3.0.8, and would appreciate some love ... ;-)
__________________
"3.4 is much better than 3.1, 3.2 or 3.3. It still is easier than 3.0.9, but it is more convenient to play without being ridiculously easy, so it is my new favorite of the versions." - Timo Pietila
Magnate is offline   Reply With Quote
Old September 5, 2011, 13:34   #10
fph
Knight
 
Join Date: Apr 2009
Location: Berlin / Italy
Posts: 776
fph is on a distinguished road
I'll try - but is the information they contain up-to-date?
__________________
Dive fast, die young, leave a high-CHA corpse.
--
You read a scroll labeled 'lol gtfo' of Teleport Level.
fph is offline   Reply With Quote
Reply


Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)
 
Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

Similar Threads
Thread Thread Starter Forum Replies Last Post
Pref files relic Vanilla 11 August 9, 2011 20:40
Help with dubtrain sound files please Magnate Vanilla 8 December 31, 2010 22:19
Updating help files fizzix Vanilla 23 August 3, 2010 04:19
delete.me files konijn_ Development 2 December 8, 2009 22:43
edit files? tummychow Vanilla 2 September 13, 2009 20:38


All times are GMT +1. The time now is 13:44.


Powered by vBulletin® Version 3.8.11
Copyright ©2000 - 2019, vBulletin Solutions Inc.