I think I will commit the changes already done. It is almost perfect now, and we can always improve it during the release.

Thank you for cooperation and see you

/Joe

I think we leave it as it is. As you said mixing languages in graphics is not that easy..different fonts, sizes and directions as well.
Meanwhile I will keep experimenting and if I found something useful I will come back to the forum.

/rafat

Joe

Reports look good now.. Where are the fonts size in the graphics of the Top 10 Customers are defined?

Thanks

The graphics for balance-sheet, pie, donut and half donut shold now be ok.

The graphics for sales customers top 10 is not changed, but you could try to reduce the MAXLEN in the define on line 63.
It is currently 27.
I see a big fontsize on the graphics, so try with 23 or so.

This is difficult to do when you mix Arabic and English language in the same graph.

But try to experiment a little.

/joe

The Dashboard seems fine for LTR. A little buggy with RTL. Let me take Arabic as an example as am testing that language.
Now I am in Preferences and chose Default theme and Arabic Language:
Sales --->Dashboard  Top 10 Customers
Arabic Customer Names are Truncated to fit---OK
English Customer Names are not Truncated---Overlap
Please note we do mix Arabic and English -- I dont know if this is a problem and we should not.
Only in pie or donut.

As for the reports definitely there is an overlap for Donut and Pie only for Balance and PL sheets.

See attached.

I have now gone through all the graphics on both SVG and traditional Graphics (JPG, PNG etc.).

The Graphic engine will truncate all labels longer than 27 chars in all the graphics. This make it much more flexibel to work with.

All the Graphics are now unified and will work as you want it to do.

Download the attached and replace it in /reporting/includes/class.graphic.inc and /includes/dashboard.inc folders.

You might try the reports Customer Aged.. and Supplier Aged (try the various graphic types). Also Balance Sheet and PL Sheet.

There was a reported bug under 'Report Bugs here'. Have a look here, there are 2 new files that should fix the issues here.

See ya Monday

/Joe

I only had to clear my browser cache, then it worked with removing the table class in default.css.
So now we should just fix the default.css and it should be ok. Cool . Good work, @poncho1234!!
Anyhow it is good to also test that the LTR still works everywhere.

Ok, you still have some problem with your test. I will have to look at this. Any thing to reproduce this? Long text or so?

We are celebrating the Mid Summer today, so don't expect to much work here

/Joe

I am glad that you, @poncho1234, is contributing in this important matter.

You wrote that eliminating the table class in default.css would do it automatically. I couldn't reporduce this, will you upload your default.css, so I can test this.

Regarding the Graphics Legends in class.graphic.in . It should now work both in LTR and RTL. Please replace the attached file in /reporting/includes folder and let me know.

@rafat

It surely looks better now

/joe

As per https://www.w3.org/ the default text alignment for LTR is left and for RTL is right.

So removing from File:default.css

32: table {
33:     text-align: left;
34: }

Will give control of text alignment back to HTML as it probably should be, giving the correct alignment for both LTR and RTL.

Further reference:https://www.w3.org/TR/html401/present/graphics.html

and

https://www.w3.org/TR/html401/struct/tables.html

/Joe