Fix display bug when not showing summary/holdings.

achannarasappa / ticker

Terminal stock ticker with live updates and position tracking

GNU General Public License v3.0

4.94k stars 264 forks source link

Fix display bug when not showing summary/holdings. #222

Closed tomclegg closed 2 years ago

tomclegg commented 2 years ago

Extra newline caused view to be N+1 rows on an N-row screen, which is not handled well by bubbletea.

tomclegg commented 2 years ago

using 80x25 terminal and example config:

compact: true
show-tags: false
show-separator: false
interval: 1
watchlist:
  - CM
  - CM.TO
  - BMO
  - BMO.TO
  - RY
  - RY.TO
  - TD
  - TD.TO
  - NA.TO
  - BNS
  - BNS.TO
  - AAPL
  - GOOG
  - MSFT
  - AMZN

running v4.5.1:

running with the fix:

achannarasappa commented 2 years ago

This looks great! Thanks for this fix