Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Bootstrap-UI #354

Closed
wants to merge 21 commits into from
Closed

New Bootstrap-UI #354

wants to merge 21 commits into from

Conversation

tamaskan
Copy link

Updated JQuery
Included Standard Roboto-Font
Custom Date-Format via http://momentjs.com/
Coin-Icon-Font via https://labs.allienworks.net/icons/cryptocoins/
Added Total Account-Value and Value per Coin
Different collapsible Sections and Filters
Progressbars and visual Min-Coin-Warning
Increased max. Update-Interval to 3600

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [N/A] I have read CONTRIBUTING.md
  • [N/A] I fully understand Github Flow.
  • My code adheres to the code style of this project.
  • [N/A] I have updated the documentation in /docs if I have changed the config, arguments, logic in how the bot works, or anything that understandably needs a documentation change.
  • [N/A] I have updated the config file accordingly if my change requires a new configuration setting or changes an existing one.
  • [N/A] I have tested the bot with no issues for 24 continuous hours. If issues were experienced, they have been patched and tested again.

Updates JQuery
Roboto-Font
Custom Date-Format via moment.js
Coin-Icon-Font via https://labs.allienworks.net/icons/cryptocoins/
Total Account-Value and Value per Coin
Different collapsible Sections and Filters
Progressbars and visual Min-Coin-Warning
Increased max. Update-Interval
@tamaskan tamaskan mentioned this pull request May 24, 2017
@rnevet
Copy link
Collaborator

rnevet commented May 24, 2017

General comment, use spaces (not tabs), keep things correctly aligned.

@laxdog
Copy link
Collaborator

laxdog commented May 25, 2017

Do we need all the binary files? It's generaly a bad idea to commit binaries to git w/o annex or something similar.

@tamaskan
Copy link
Author

I haven't found a hosted version of the cryptocoin-font, but it could be reduced to a single .woff2-file

Reduced the CryptoCoin-Font to a single .woff2-File
Show the amount of earned Coins since first UI-Start (Stored in
localstorage)
@rnevet
Copy link
Collaborator

rnevet commented May 25, 2017

'Show the amount of earned Coins since first UI-Start (Stored in
localstorage)'

Not needed, we are going to have complete lending history in this branch:
#349

Please correct tabs as it's really hard to review the code this way.

@rnevet
Copy link
Collaborator

rnevet commented May 25, 2017

While I find displaying the opened loans interesting, displaying the cancel loan events is useless as this happens on every bot cycle and doesn't actually bring any information.

@rnevet
Copy link
Collaborator

rnevet commented May 25, 2017

image

@rnevet
Copy link
Collaborator

rnevet commented May 25, 2017

The filtering buttons are ignored when a reload happens. :)

@rnevet
Copy link
Collaborator

rnevet commented May 25, 2017

I like the overall design, however I think many of the features should be discussed.

In general I would prefer to have first design change and then separate PRs for features... this will also make it easier to review and approve.

@tamaskan
Copy link
Author

The creators of this bot are free to use only the non-controversial parts and open issues for discussions :-)

var totalvalue = 0;

//Min Coins based on https://github.com/BitBotFactory/poloniexlendingbot/issues/347
var items = [['BTC','BTS','CLAM','DASH','DOGE','ETH','FCT','LTC','MAID','STR','XMR','XRP'],[0.00019124,10,10,0,100,0,100,0,10,100,0,100]];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bitcoin min is actually 0.01, in the issue he was demonstrating the different values of the minimums.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a list of these minimums ?

@kingo55
Copy link

kingo55 commented May 28, 2017

The design changes look really nice!

Hiding a bunch of stuff under accordions makes this design less practical though. I like to be able to view all the stats upon a glance. If I have to tap to open accordions, it takes time and effort. Arguably not much, but enough that it makes the old design more useful at this stage.

It gets really annoying when the bot refreshes the page and you lose all the accordion states! :(

@tamaskan
Copy link
Author

I can change that behavior, thanks for the feedback :-)

@rnevet
Copy link
Collaborator

rnevet commented Jun 26, 2017

btw, I made a JSFiddle for prettyFloat
https://jsfiddle.net/mut72zcp/

@Evanito Evanito requested a review from rnevet July 26, 2017 01:19
@rnevet
Copy link
Collaborator

rnevet commented Aug 1, 2017

@tamaskan how about making this an alternate theme based on:
86c3521
?

@mchl18
Copy link
Contributor

mchl18 commented Aug 1, 2017

some friends and i will work on an alternative frontend soon, but probably on my fork. It really needs an overhaul. I wanna try if vue.js could work and maybe replace bootstrap

@rnevet
Copy link
Collaborator

rnevet commented Aug 1, 2017

@mchl18 have you seen the option to create alternate themes?

@tamaskan
Copy link
Author

tamaskan commented Aug 1, 2017

@rnevet sure, that is not a problem.
@mchl18 awesome :-)

@mchl18
Copy link
Contributor

mchl18 commented Aug 1, 2017

also wanna try and write a config interface...wip

@mchl18
Copy link
Contributor

mchl18 commented Aug 1, 2017

@rnevet no i have not. I just fiddled with some css overlay but i think i want a complete overhaul

@tamaskan
Copy link
Author

tamaskan commented Aug 1, 2017

thought about a configuration-interface, but the main problem is to keep it up to date

@mchl18
Copy link
Contributor

mchl18 commented Aug 1, 2017

i have some ideas and will try it on my fork

@tamaskan
Copy link
Author

tamaskan commented Aug 1, 2017

great :-) do you need help ?

@mchl18
Copy link
Contributor

mchl18 commented Aug 1, 2017

I havent had time to set it up, wanna meet with some friends to discuss, if I run into questions i will let you know

@mchl18
Copy link
Contributor

mchl18 commented Aug 1, 2017

vue.js seems tempting but idk if it can handle the whole parsing and all, havent used it before and really wanna try it. also i wanted to create tiles for the currencies instead of tables to make it more customisable and responsive. also bulma could be a nice alternative to bootstrap since I like flexbox.
But i will make a new issue and PR for this. Have to work full time right now, we'll see if that works out. I am a frontend guy by heart

@tamaskan
Copy link
Author

tamaskan commented Aug 2, 2017

never worked with that, but it should be able to handle that.

@mchl18
Copy link
Contributor

mchl18 commented Aug 2, 2017

yeah it looks really slim and easy to set up, much better than angular and simpler than react, but also only js at its core

@utdrmac
Copy link
Contributor

utdrmac commented Aug 27, 2017

@tamaskan What's the status on this? There are some other PRs that want to make UI changes too but this is holding that up and will make the merge of this PR harder. Are you pausing this effort for the moment?

@tamaskan
Copy link
Author

yes, i am still working on this. the bot supports multiple page-templates, so i see no reason why another developer can't create a PR with his Idea.

@rnevet
Copy link
Collaborator

rnevet commented Aug 27, 2017

@utdrmac as @tamaskan mentioned a recent change to the bot was allowing multiple html templates (IMO might be harder to maintain but it is in).
86c3521

If this is regarding adding the graphs view, I would be happy to have it in the current template.

@gincarbone
Copy link

great job

@bkuhl
Copy link

bkuhl commented Jan 7, 2018

Seems like a while since this has had a contribution. Is this still a work in progress? Can a screenshot be provided?

@BitBotFactory BitBotFactory deleted a comment from gincarbone Feb 7, 2018
@bkuhl
Copy link

bkuhl commented Apr 25, 2018

Still no updates here.... hrmmmm... what do we do with this PR?

@utdrmac
Copy link
Contributor

utdrmac commented Jul 6, 2018

Is this PR dead?

@SPIRY-RO
Copy link

I do think it deserves a neat UI. And the initial PR had a very strong idea in that direction.

@tamaskan
Copy link
Author

i switched to coinlend.org, but everyone is free to use the code and create something beautiful

@tamaskan tamaskan closed this by deleting the head repository Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.