mifort-org / mifort-timesheet

Mifort Timesheet
http://mifort.org/timesheet/
Apache License 2.0
30 stars 19 forks source link

Report: Print: comments are cut #461

Open anabenko opened 7 years ago

anabenko commented 7 years ago

win 7, FF 55.0.3 role: all dev

  1. Enter long comment in Timesheet
  2. Open Reports (set filters to include record with long comment)
  3. Print -> comment is cut comm
AlexKolonitsky commented 6 years ago

image

AndrewRest commented 6 years ago

Why do we need to hide the top "Aggregation By" section? How can I use it if we hide it?

About comment: It's aggregated field as well. How it will looks for Employee report? Hundred lines? For me we need to improve our current approach - allows user to see it separatelly.

AlexKolonitsky commented 6 years ago

It a print version of the page: 1) we shouldn't display useless controls here like "Aggregation By" and pagination (it is better to have simple text "page x from y") 2) About comments: We should show as much info as possible, we can limit text length to reasonable value like 100-200 symbols and end with "..." in case it is longer @AndrewRest what do you think?

altrofimov commented 6 years ago

@AndrewRest its only for the Print. You choose what you want to print and on print.css we hide the controls that are not bringing any value for the printed version.