You are viewing a single comment's thread from:

RE: SteemAX v0.5 Release and New Features ~ SteemAX.info Now Up And Running!

in #utopian-io6 years ago (edited)

Thank you for your contribution.

  1. document.getElementById is quite outdated i.e. you may want to try Jquery or other JS framework e.g. document.getElementById("memo"+itemshowing).style.display='none' could be just replaced by something like $('memo'+itemshowing').hide()
  2. no comments at all, which is really hard to follow.
  3. it is not a good practise to declare a variable e.g. var something; and then initialize it later.
  4. The coding quality is really disappointing - and quite old fashion.

Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.

To view those questions and the relevant answers related to your post, click here.


Need help? Write a ticket on https://support.utopian.io/.
Chat with us on Discord.
[utopian-moderator]

Sort:  
Loading...

Also, the only thing not commented is the website (HTML and Javascript because this will all be minified and impossible to read anyway). You seemed to be very focused on my javscript code, but apparently have nothing to say about the CSS, HTML or Python I wrote for this new feature. Where is your feedback on those languages?

Loading...

Coin Marketplace

STEEM 0.19
TRX 0.15
JST 0.029
BTC 63407.49
ETH 2645.11
USDT 1.00
SBD 2.81