j-diefenbach
commented
6 months ago Would definitely have to consider the icon spam / clutter. E.g. if you get a new pokemon vs a new nature are they really the same priority. And also gaining one new nature or candy is probably very low info. I think some of these could be good but would have to make sure you're not going overboard or the option to turn off certain notifs, and will definitely need some way to clear notifs with that though be conscious of using too many buttons / settings.
Proposal: NEW Icons over pokemon with improved IVs , new moves, rarities, candy thresholds.
Where: The "Pokedex" menu, top-right side of each pokemon and top right side of each generation Some duplicate "NEW" tags over their corresponding location
Why: It can be hard to track new anything, I couldn't remember where by new ability was located, and I had to go through everything to find it.
How: Flags can be a single variable attached to each pokemon in the pokedex maybe called viewed? Flags are an enum with the following: none = 0 NEW = 1 //When the pokemon is new , hidden when hovered IVs = 2 //When updated IVs are available, hidden when IVs viewed MOV = 4 //When a new egg move is available , hidden when hovered SNY = 8 //When a new shiny level is available , hidden when hovered CDY = 16 //When candies gained is over the minimum threshold for an available purchase, hidden on purchase if candies < available purchases ABI = 32 //When a new ability is present , hidden when cycled NAT = 64 // When a new nature is present , hidden when cycled
Any invalid values can be interpreted as 0 or 127 to prevent errors. (anger anyone with OCD by setting the value to 127 by default)
Flags can be public, as they have no gameplay effects.
Suggestions on the best way to save flag changestates? Can't exclusively be saved with run data.
flags could be set with the following functions in the appropriate file:
set_viewed_new(pokemon_id , enum) pokedextable.pokemon_id.viewed = pokedextable.pokemon_id.viewed | enum set_viewed_old(pokemon_id , enum) pokedextable.pokemon_id.viewed = pokedextable.pokemon_id.viewed & !enum (need confirmation, but pretty sure it's good code) pokedextable.pokemon_id is a palceholder, advise on correct place to insert this flag as a property