DestinyItemManager / DIM

Destiny Item Manager
https://destinyitemmanager.com
MIT License
2.07k stars 644 forks source link

Weekly Activity #145

Closed ChuckTerry closed 7 years ago

ChuckTerry commented 9 years ago

How about a weekly activity mouseover or dropdown? Quick example below thrown together in paint. Something that would be convenient, but not always shown.

quick example

SunburnedGoose commented 9 years ago

I like the idea, but it is outside the scope of an item manager atm. There is no bandwidth for this feature.

LongLiveCHIEF commented 9 years ago

@SunburnedGoose I'll work on a PR for this if you're up for it. I agree it's not in scope, but I'd hate to make a second extension.

ChuckTerry commented 9 years ago

@LongLiveCHIEF I was considering trying to. But html & chrome are outside my programming knowledge. just wanted a reason to remove destinystatus from my bookmarks bar.

SunburnedGoose commented 9 years ago

Building a PR is fine. Just work with us upfront on your design and show me your repo where you're working. I wouldn't want you to build something and then have to rework it to make it fit.

Thanks for helping out.

MattWhilden commented 9 years ago

From a broader level, is the goal of DIM to replace all my other bookmarks and become a one stop shop?

DrewScoggins commented 9 years ago

I have to agree with @MattWhilden on this. It seems like something like this is outside the scope of what DIM is. I much prefer having a suite of tools that all do their job extremely well rather then clunky tool that tries to be everything to everyone. Just my .02

SunburnedGoose commented 9 years ago

The DIM team is focused on Item Management and related activities. Now what's a related activity?

Just a few ideas for what we can add to DIM. I don't think this is taking over Destiny Public Events,

kyleshay commented 9 years ago

if there is a nice way to fit if in the tool without it being obtrusive then i'd say go for it. i'd always like for items to be forefront, but if we can sneak a menu option somewhere that's totally cool. (similar to how the loadouts dropdown work- unobtrusive, but easily accessible.)

if you do go this route, before commiting to all of the work, pass around some mockups on what you're thinking. we can iterate on them and then get them implemented!

otherwise, feel free to get the code done just don't expose it anywhere, and we can work on the view toward the end.

also, keep mobile in mind.

bhollis commented 8 years ago

IMO Bungie.net handles this info just fine. Post-TTK, do we still want to do this?

Mulchman commented 8 years ago

destinystatus.com and destinchecklist.net do a terrific job tracking this stuff as well...

On Thu, Jan 21, 2016 at 9:10 PM, Ben Hollis notifications@github.com wrote:

IMO Bungie.net handles this info just fine. Post-TTK, do we still want to do this?

— Reply to this email directly or view it on GitHub https://github.com/DestinyItemManager/DIM/issues/145#issuecomment-173810622 .

SunburnedGoose commented 8 years ago

Move it to v4. On Thu, Jan 21, 2016 at 11:18 PM Mulchman notifications@github.com wrote:

destinystatus.com and destinchecklist.net do a terrific job tracking this stuff as well...

On Thu, Jan 21, 2016 at 9:10 PM, Ben Hollis notifications@github.com wrote:

IMO Bungie.net handles this info just fine. Post-TTK, do we still want to do this?

— Reply to this email directly or view it on GitHub < https://github.com/DestinyItemManager/DIM/issues/145#issuecomment-173810622

.

— Reply to this email directly or view it on GitHub https://github.com/DestinyItemManager/DIM/issues/145#issuecomment-173813502 .

kyleshay commented 8 years ago

linking: #254 since they use the same API call and return the same data. doesn't need to be implemented at the same time, but code will be heavily shared.

SunburnedGoose commented 8 years ago

Should work closely with DSR on this. On Fri, Jan 22, 2016 at 10:49 AM kyle shay notifications@github.com wrote:

linking: #254 https://github.com/DestinyItemManager/DIM/issues/254 since they use the same API call and return the same data. don't need to be implemented at the same time, but code will be heavily shared.

— Reply to this email directly or view it on GitHub https://github.com/DestinyItemManager/DIM/issues/145#issuecomment-173974659 .

ChuckTerry commented 8 years ago

I didn't really expect this to be revived, however my raid group will be extremely pleased to see this added. I just wish I had the programming experience to help. Btw guys, great job on where DIM has went from conception, we recommend DIM to everyone now, keep up the good work :)

kyleshay commented 7 years ago

Closing in favor of #1886

kyleshay commented 7 years ago

@tda0909 let us know what you think! :)

ChuckTerry commented 7 years ago

I think that I've needed this in my life for years ;) it's perfect, and should work well for destiny 2... Plus it looks much better than my paint mockup from a couple years ago :)

Good work guys, I'm really glad to see this finally come to fruition, keep up the amazing work.

kyleshay commented 7 years ago

ha :D thanks for the feedback, agreed it will be useful for D2!!