ESI-FAR / INA-tool

A tool for visualising and analysing Institutional Statements
MIT License
2 stars 0 forks source link

Increase font size #71

Closed ermannolocascio closed 3 months ago

ermannolocascio commented 3 months ago

At the moment this resulted to be not very readable.

ermannolocascio commented 3 months ago

Following I made some renders of the flowchart with font at 10px (the original), 12px, 13px and 14px.

Please, let us know which is the preferred one. Thank you :)

10px ---------------------- 20240718-111711_Institutional_statement_template

12px---------------------- 20240718-113426_Institutional_statement_template_12px

13px---------------------- 20240718-113537_Institutional_statement_template_13px

14px---------------------- 20240718-114248_Institutional_statement_template_14px

aminehghorbani commented 3 months ago

Dear Ermanno, in my view they are still not readable. Would it be possible to decrease the margin of the text in the shapes? Also maybe increase the space between the lines? When you make them bigger, they become more unreadable because the space between the lines become smaller

ermannolocascio commented 3 months ago

Thank you @aminehghorbani for your feedback. I'm trying to find a good compromise. How about this? The space between lines has been set to 1.25 and the margins are reduced (we reduce the text lenght for each line, this is how it works. Now it is set to 23 characters, before was 25). The font size now is 12px. Please, feel free to ask for changes. We can iterate it. If you prefer we do it via teams in call.

20240718-133306_Institutional_statement_template

aminehghorbani commented 3 months ago

Thanks Ermanno for the new update. Its much better, but it's still not easy to copy/paste a diagram into a word file because it's still relatively unreadable. Can the text be fluid in the shape? I.e., depending on the amount of text, it changes sizes to fit the shape? Reducing the margin might help further or not? and maybe a font other than the one you have can improve readability? But in general, it's much much better, so if it doesn't work it can be an acceptable solution for now.

ermannolocascio commented 3 months ago

Hi Amineh,

the best thing we can do at this stage is to go for this compromise for now given that you need to be able to tune the font size depending on the statement text you upload. This would allows you to achieve the right render. I open an issue on this to be solved for the next future (#79).

Please, note that Sander (@sjvrijn) will have to accept the pull request (#78) before you see the changes (the increased font size) take effect.

aminehghorbani commented 3 months ago

Hi Ermanno, Thanks for the update. I don't seem to find the link to the website anymore, can you paste it here please?

Best wishes, Amineh