From 193baf36a02cc248fa1457aa9f9374994e7665fd Mon Sep 17 00:00:00 2001
From: igoradamenko <mail@igoradamenko.com>
Date: Mon, 26 Mar 2018 14:30:08 +0300
Subject: [PATCH] rewrite todos

---
 web/TODO.md                | 18 ++++++++++++++----
 web/app/common/promises.js |  5 -----
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/web/TODO.md b/web/TODO.md
index 5d40fa50..750cca66 100644
--- a/web/TODO.md
+++ b/web/TODO.md
@@ -1,12 +1,23 @@
+major fixes:
+
+- comment scroller:  
+  we can't get access to window.parent.location, 
+  so we need to use postMessage for pushing location.hash 
+  into the iframe 
+
 major features:
 
 - improve design
   - check mobile ui
-  - add styles for md text
+  - check styles for md text
   - add time format 'X hours ago'
+  - try to move voting buttons from the top line to the left side
+  - improve thread design
 - edit comment
   - `PUT /api/v1/comment/{id}?site=site-id&url=post-url`
 - add description of web part to readme
+- show comments of blocked users in the list of them
+
 
 optimizations:
 
@@ -17,10 +28,9 @@ optimizations:
   
 minor features:
   
-- add manual sort
+- maybe add manual sort
   - `GET /api/v1/find?site=site-id&url=post-url&sort=fld&format=tree`
-- add comments counter
-  - `GET /api/v1/count?site=site-id&url=post-url`
+- show list of user comments for admin?
 
 
 other:
diff --git a/web/app/common/promises.js b/web/app/common/promises.js
index cd77d551..95885564 100644
--- a/web/app/common/promises.js
+++ b/web/app/common/promises.js
@@ -1,11 +1,7 @@
 import Promise from 'promise-polyfill';
 
-/* eslint-disable no-underscore-dangle */
 Promise._unhandledRejectionFn = () => {};
-/* eslint-enable no-underscore-dangle */
 
-// TODO: need to figure out, do we really need finally?
-/* eslint-disable no-extend-native */
 Promise.prototype.finally = function finallyFn(callback) {
   const constructor = this.constructor;
 
@@ -14,6 +10,5 @@ Promise.prototype.finally = function finallyFn(callback) {
     (reason) => constructor.resolve(callback()).then(() => reason)
   );
 };
-/* eslint-enable no-extend-native */
 
 window.Promise = Promise;