I was about to ask you if I you had an objection if I reformat the files to
2 spaces (what I prefer).  But if you want to stick to 4,  i'll reformat
all to 4 and follow that. Let me knoe
On Apr 11, 2014 1:57 PM, "Xavier Antoviaque" <notifications@github.com>
wrote:

> In mentoring/public/js/questionnaire.js:
>
> > +
> > +      // Set the width/height
> > +      var tip = $('.tip', popupDOM)[0];
> > +      var data = $(tip).data();
> > +      if (data && data.width) {
> > +        popupDOM.css('width', data.width)
> > +      } else {
> > +        popupDOM.css('width', '')
> > +      }
> > +
> > +      if (data && data.height) {
> > +        popupDOM.css('height', data.height);
> > +      } else {
> > +        popupDOM.css('height', '')
> > +      }
> > +
>
> Btw in this file (and maybe others?) you have some spacing issues -
> sometimes it's indented by 2 spaces instead of 4, and some of the lines
> aren't aligned.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/edx-solutions/xblock-mentoring/pull/21/files#r11543882>
> .
>