Reviewer comments overlapped

Software Support
1

Hi,

I’m using OJS 3. The comments from reviewer “for author and editor” and “for editor only” appear to be overlap in the “read review” function on the “Review” page. What can I do to make it display properly so that the editor can read properly and make an informed decision.

Thanks

Choo

2

Hi @ymchoo,

Could you include a screenshot? What exact version of OJS are you using?

Regards,
Alec Smecher
Public Knowledge Project Team

3

Hi

We are using version 3.0. And here’s the screenshot.

Thanks and regards

Choo

4

Hi @ymchoo,

I suspect this is related to the CSS styling of one of the fields of that page, e.g. reviewer guidelines. I’d suggest using a tool like Chrome’s “Inspect” or Firebug for Firefox to inspect the styles of those elements. It may be that you’ve pasted something incompatible into a TinyMCE field.

Regards,
Alec Smecher
Public Knowledge Project Team

5

Hi Alec,

Our technical team have checked the styling using Chrome’s “inspect”. Below
is our findings.

The Read More button triggered the revealMore javascript function. However
it appeared that the element has a max-height of 192px. If the styling is
removed, display seemed to be OK. The review content itself contained just
plain paragraph. Please advise on the best way to fix this. Thank you.

With max-height styling:

Without max-styling:

Source code for Review content:

Best regards

Choo

6

Hi @ymchoo,

That max-height settings hould get removed when the Read More button is clicked. It looks like that’s not happening for some reason. Can you tell me what OS (eg - Windows 7 or 10, Mac) and browser (eg - Internet Explorer 10/11/Edge, Chrome, Firefox) you’re using? I’ll see if I can reproduce the issue with your OS/browser combination.

7

Hi,

Our technical team has rectified the problem. The solution is:

A line change was done in file RevealMoreHandler.js

The line
this.getHtmlElement().removeClass(‘isHidden’).css(‘max-height’, ‘auto’);

was changed to
this.getHtmlElement().removeClass(‘isHidden’).removeAttr( ‘style’ );

Best regards

Choo

8

Hi @ymchoo,

That sounds like a good solution. I’ve never been able to reproduce the problem myself. Can you tell me any information abotu what OS or web browser you were using when you experienced the problem?

9

Hi

We are using OJS 3.0, and using chrome browser.

rgds

Choo