[review-together] Cleanups to scroll position work
Review Request #6884 — Created Feb. 1, 2015 and submitted
Peter Tran did a lot of preparatory work for showing the scroll
position of your peers in a TogetherJS session. This work was
landed in a project branch, awaiting some clean-up once the term
was over.This is the clean-up I had in mind. I've fixed the following:
* Scroll position indicators now have a high z-index to avoid being
hidden behind other elements.
* If no hub URL has been specified, properly falls back to the Mozilla
TogetherJS hub.
* Moves the ReviewTogether JS code into a closure to avoid polluting the
global namespace.
* If a peer's colour changes during a session, this is now manifested
for everybody in the session without requiring a page reload.
* Switched a lot of double-quotes to single-quotes.
* Leaving a TogetherJS session removes all scroll position indicators.
* When somebody leaves a session, their scroll position indicator is
removed.
Installed the extension locally. Then, viewed a long diff in a local review request.
Clicked on the 'Chat' link in the header to open the TogetherJS interface. Got the
sharing link from TogetherJS.Then, in a second browser, went to the sharing link. I was able to view the scroll
position for both browser sessions on the long diff.
- Description:
-
Peter Tran did a lot of preparatory work for showing the scroll
position of your peers in a TogetherJS session. This work was landed in a project branch, awaiting some clean-up once the term was over. This is the clean-up I had in mind. I've fixed the following:
~ - Scroll position indicators now have a high z-index to avoid being hidden behind
other elements.
~ - If no hub URL has been specified, properly falls back to the Mozilla TogetherJS hub.
~ - Moves the ReviewTogether JS code into a closure to avoid polluting the global namespace.
~ - If a peer's colour changes during a session, this is now manifested for everybody
in the session without requiring a page reload.
~ - Switched a lot of double-quotes to single-quotes.
~ * Scroll position indicators now have a high z-index to avoid being
~ hidden behind other elements. ~ * If no hub URL has been specified, properly falls back to the Mozilla ~ TogetherJS hub. ~ * Moves the ReviewTogether JS code into a closure to avoid polluting the + global namespace. + * If a peer's colour changes during a session, this is now manifested + for everybody in the session without requiring a page reload. + * Switched a lot of double-quotes to single-quotes. + * Leaving a TogetherJS session removes all scroll position indicators. + * When somebody leaves a session, their scroll position indicator is + removed. - Scroll position indicators now have a high z-index to avoid being hidden behind
- Commit:
-
6e2e0999b17f6f9868a80c6fa2945b6ed0cfe5fc3341ad25920fd918117f61da78588d04f598c5d7
- Diff:
-
Revision 2 (+87 -94)
-
Tool: PEP8 Style Checker Processed Files: review_together/review_together/extension.py Ignored Files: review_together/review_together/static/js/togetherjs.js review_together/review_together/static/js/review-together.js review_together/review_together/static/js/together.js review_together/review_together/static/css/review-together.less Tool: Pyflakes Processed Files: review_together/review_together/extension.py Ignored Files: review_together/review_together/static/js/togetherjs.js review_together/review_together/static/js/review-together.js review_together/review_together/static/js/together.js review_together/review_together/static/css/review-together.less
- Commit:
-
3341ad25920fd918117f61da78588d04f598c5d7197aa31ea74830c05ba29df905e0857acb97b83b
- Diff:
-
Revision 3 (+89 -94)
-
Tool: PEP8 Style Checker Processed Files: review_together/review_together/extension.py Ignored Files: review_together/review_together/static/js/togetherjs.js review_together/review_together/static/js/review-together.js review_together/review_together/static/js/together.js review_together/review_together/static/css/review-together.less Tool: Pyflakes Processed Files: review_together/review_together/extension.py Ignored Files: review_together/review_together/static/js/togetherjs.js review_together/review_together/static/js/review-together.js review_together/review_together/static/js/together.js review_together/review_together/static/css/review-together.less