Bug 486162

"svn co http://svn.fedorahosted.org/svn/selinuxguide/community/trunk/SELinux_User_Guide"

Note: Makefile currently has "OTHER_LANGS = ar-AR" commented out.

I have this change in place, but I can't test it until po2xml is fixed in rawhide.

Please confirm that the attribute to be set is 'language' and not 'lang'.

'language' is not a valid attribute in DocBook and setting that attribute will prevent a book from building as publican will not process input files that are not valid DocBook XML.

yes, sorry I meant lang

thanks,

Publican now emits lang="xx-YY" to intermediate XML.

This patch will be pushed to rawhide when the translations of the common content are complete.

could you please point me to the version-release so that I can test it

http://koji.fedoraproject.org/koji/buildinfo?buildID=91336

does not mention anything in %changelog
just the mass rebuild

(In reply to comment #6)
> could you please point me to the version-release so that I can test it
> 
> http://koji.fedoraproject.org/koji/buildinfo?buildID=91336
> 
> does not mention anything in %changelog
> just the mass rebuild  

As I said "This patch will be pushed to rawhide when the translations of the common content are complete.  "

The translations are not complete, so the new package has not been pushed to koji yet.

if you mean adding ar-AR to
http://svn.fedorahosted.org/svn/publican/trunk/publican-fedora/

I can do the translation

tell me to do that if I'm right

FYI publican 0.44 and publican-fedora 0.18 have been pushed in to koji and built OK. They should get pushed in to rawhide in the next push.

http://koji.fedoraproject.org/koji/buildinfo?buildID=93868
http://koji.fedoraproject.org/koji/buildinfo?buildID=93869

Cheers, Jeff.

I have done some translations and attached a minor fix to the Arabic css file

could you please apply the attached patches
one on publican and one on publican-fedora

then fix the RTL images in publican by 

mkdir -p content/common/ar-AR/images
for i in stock-go-back.png stock-go-forward.png
do
  convert -flop content/common/en-US/images/"${i}" content/common/ar-AR/images/"${i}"
done

and in publican-fedora by

mkdir -p ar-AR/images
for i in stock-go-back.png stock-go-forward.png
do
  convert -flop en-US/images/"${i}" ar-AR/images/"${i}"
done

Created attachment 335002 [details]
publican common content translation and rtl enabled css

patch -p1

Created attachment 335004 [details]
publican fedora ar translation

patch -p1

Created attachment 335005 [details]
screenshot of fedora's selinux user guide

Created attachment 335058 [details]
New langauge override css

Hi, I applied all the patches except the css patch.

For the css I restructured it to minimise the amount of duplication. Duplicating the content in the brands was enough to make it unmaintainable before, doing that for the languages would be a nightmare.

What I did was :

1: add a new include to default.css to include lang.css
2: make a delta of the changes in common.css to produce lang.css for ar-AR
3: switch next/previous images in lang.css, avoiding manipulating the images.
3: test on SELinux_User_Guide

It looks good to me ... admittedly I can't read the content, but it seems to work correctly, including the next/previous buttons being switched.

Taking this approach really reduces the maintenance overhead, especially when other languages wish to modify things. It also means that all brands automatically get this these fixes for ar-AR since brands do not over ride default.css and as lang.css is included last it will override all other customisations.

Screen shot to follow!

Created attachment 335059 [details]
Screen shot of styled HTML.

Thanks for your patches and translations Muayyad :)

Great!
nice work and good idea
the screenshot is perfect

but I wonder can't we find a better way to put admonition images [note/warning..etc.] to the right other than background-position

because I set it to be right but it should be right-1em to make a small 1em gap

but of course having 0em padding is better than having them on the left

publican-fedora-0.18-0.fc10,publican-0.44-0.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/publican-fedora-0.18-0.fc10,publican-0.44-0.fc10

publican-fedora-0.18-0.fc10, publican-0.44-0.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.