rubensayshi / gw2spidy

GW2Spidy - Guild Wars 2 Tradingpost Graphs
http://www.gw2spidy.com
231 stars 86 forks source link

Change legend to allow for enough space for more lines #38

Open rubensayshi opened 12 years ago

rubensayshi commented 12 years ago

The legend should be able to hold multiple items and even more then the 4 lines (raw, daily, weekly, monthly moving average) somehow.

I was thinking of moving it into a div in the sidebar for example, probably completely writing custom code for it instead of the highcharts default so that we can prettify it ;-)

timeinfeldt commented 12 years ago

Is it possible with the standard Highcharts to put the legend left or right of the chart? That would already be enough I think. The chart would move up the page a bit and you'd have a nice left-right balance between data and filters.

Customizing Highcharts will probably eat a lot of time, so I'd recommend against that ;)

rubensayshi commented 12 years ago

it has default functionality to move the legend outside of the chart in any given div you want

timeinfeldt commented 12 years ago

Then one easy way could be splitting up <div id="content" class="span8 main-content"> into a span3/span5 or span2/span6 combo for chart and legend.

Also, I believe that the <div class="well"></div> in there isn't needed, you should be able to just add the "well" class to the main-content div.

So the end result would be something like

<div id="content" class="span8 main-content well">
    <div class="span5">{{chart}}</div>
    <div class="span3">{{legend}}</div>
</div>