search

cppisking / ffrk-inspector

Issues tracker for FFRK Inspector
10 stars 8 forks source link

Item Search showing unavailable events #64

Closed BaconCatBug closed 9 years ago

BaconCatBug commented 9 years ago

Especially with the "I want to help" filter, it shows a lot of entries from the Fire Powered Ship. These should be removed once the event is finished IMHO, or at least hidden in case the event shows up again.

cppisking commented 9 years ago

I will try to come up with a way to do that, but it's not very easy On Sat, Jun 6, 2015 at 6:12 AM BaconCatBug notifications@github.com wrote:

Especially with the "I want to help" filter, it shows a lot of entries from the Fire Powered Ship. These should be removed once the event is finished IMHO.

— Reply to this email directly or view it on GitHub https://github.com/cppisking/ffrk-inspector/issues/64.

cppisking commented 9 years ago

Seems like this will be possible after all, hopefully I can finish this in time before I leave the country.

rsilva712 commented 9 years ago

Would a "hide event dungeons" check mark be difficult to implement?

cppisking commented 9 years ago

Not really, do you think that would still be useful though if there's already a check box to hide inactive events? I have that working locally so it'll go out with the next release. I don't want to clutter up the UI too much though unless you think it will really be useful

BaconCatBug commented 9 years ago

I think a box to hide inactive events is more than enough,

On 7 June 2015 at 02:22, cppisking notifications@github.com wrote:

Not really, do you think that would still be useful though if there's already a check box to hide inactive events? I have that working locally so it'll go out with the next release. I don't want to clutter up the UI too much though unless you think it will really be useful

— Reply to this email directly or view it on GitHub https://github.com/cppisking/ffrk-inspector/issues/64#issuecomment-109667592 .

cppisking commented 9 years ago

This is done, will be available later this week