GIA-3147 - A per task - 'It would be better if the header was loaded by default with NO text on the right hand side to indicate open/closed, and then when discussion has responded, the correct text is inserted into the header' - this is to avoid a flash on slower devices which intially shows the leave a comment message then changes to Comments Closed (where Comments are closed). To achieve this I've added a new function commentsOpen which adds the comments--open class and cascades to display this message when inserted into the DOM. This also required a change in iOS native code to trigger the commentsOpen function (I will need a similar function on Android native but I need help for this). I've also added a height to the containing div .container__header to prevent the movement of the text which occurred before when the messages changed. Finally changed the closed comments message in article view to appear below the number of Comments h2.
GIA-3147 - Corrected position and padding for container__action--sort class (for the sort order CTA) so that it doesn't shift down when the Leave a comment message/icon appears. Also I removed the debug console and log messages and corrected a number of minor warnings reported by SCSS linter.
GIA-3147 - Android specific class added to ensure display of the leave a comment icon and not to display it when comments--closed class is present. Also removed some commented out css.
GIA-3147 -- Fix merge conflict
GIA-3147 - Removed media query that displayed Closed Comments message in iPad views by default.
GIA-3147 - Fix branch merge conflicts
GIA-3147 - Fix branch merge conflicts 2
GIA-3147 Fixed moduleComments.html conflict as IDs were added for testing
Merge pull request #291 from guardian/GIA-3147

Gia 3147