FreeCAD icon indicating copy to clipboard operation
FreeCAD copied to clipboard

Spreadsheet: use 'text' instead of 'foreground' in tooltip

Open luzpaz opened this issue 1 year ago • 4 comments

Fixes #18013

luzpaz avatar Nov 20 '24 19:11 luzpaz

Is the "cell(s)" stuff actually useful here? It would be quite a bit less cumbersome if we dropped that word/punctuation blob.

chennes avatar Nov 21 '24 05:11 chennes

Is the "cell(s)" stuff actually useful here? It would be quite a bit less cumbersome if we dropped that word/punctuation blob.

I took a look at LibreOffice and they use just 'Font Color' and 'Background Color'. But in case of FreeCAD, I would leave the 'cell' there, because user can have Spreadsheet WB open with 3D view and than changing 'Background Color' would have different meaning and 'cell' gives a correct context to the tool.

If I was to change something, I would drop the 's' , so there would be 'cell' instead of 'cell(s)'. But that is just a minor thing.

semhustej avatar Nov 21 '24 09:11 semhustej

Verdict ?

luzpaz avatar Nov 21 '24 20:11 luzpaz

I agree with @chennes, but I don't see it as a required change to get this merged.

hyarion avatar Nov 23 '24 00:11 hyarion