elanthia-online / illthorn

Electron-based FE
18 stars 10 forks source link

Include spell name in title attribute for list items under active spells #69

Closed BryanSchuetz closed 4 years ago

BryanSchuetz commented 4 years ago

I've been truncating the spell name in the active-spells panel because I don't like the way it looks when it wraps. It would be nice if we also included the spell name in a title attribute for the list item so you could hover over the truncated name and see the full name.

Screen Shot 2020-07-13 at 10 32 55 AM

This would hep to distinguish between multiple ranks of the same spell (e.g. Elemental Defense I, II, III). If we think this makes sense, I'm happy to make the change in active-spells

chriscoyier commented 4 years ago

One thing to consider with the title attribute is that it has that hover/popup thing that you cannot control in any way.

An alternative might be to add a data-spell-name attribute and then use that to construct a hover effect if desired.

Stretch goal... add things like data-spell-description or data-spell-effects="+5 TD" for more optional scalability. That introduces some extra technical debt though as I assume those kinds of things change over time.

ondreian commented 4 years ago

Data attributes are not very accessible, really what I think is necessary here is probably a lookup table to show both spell number and spell name

On Mon, Jul 13, 2020, 10:59 Chris Coyier notifications@github.com wrote:

One thing to consider with the title attribute is that it has that hover/popup thing that you cannot control in any way.

An alternative might be to add a data-spell-name attribute and then use that to construct a hover effect if desired.

Stretch goal... add things like data-spell-description or data-spell-effects="+5 TD" for more optional scalability. That introduces some extra technical debt though as I assume those kinds of things change over time.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/elanthia-online/illthorn/issues/69#issuecomment-657611147, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIKHAQG53YRFNCV7CDWJRTR3MOLXANCNFSM4OYRWO6A .

BryanSchuetz commented 4 years ago

Is that kind of data available from the stream? I'm thinking number and spell-name might be...but not the kind of thing Chris is talking about. I'm assuming there's no JSON endpoint on the game server we can hit for a spells table—would we be OK scraping the wiki? That's not maintained officially is it?

chriscoyier commented 4 years ago

Data attributes are not very accessible

They aren't, but title isn't either (it's not read by Assistive Tech). If we were actually truncating the text, we'd need some kind of solution, but the text isn't actually truncated, it's all in the DOM properly.

For other spell information (the lookup table sounds very smart), we could chuck that stuff in the DOM too and hide/show it based on settings potentially.

ondreian commented 4 years ago

Maybe something like:

401 / Elemental Defense...
406 / Elemental Defense...

Or however we decide to join them, but would at least give you all the info you need in the same row, and still allow truncating

Is that kind of data available from the stream?

Not really, it's either spell numbers or spell names, you can set it with with the FLAG verb

I'm thinking number and spell-name might be...but not the kind of thing Chris is talking about.

Nah, I was just trying to think of a way to handle truncation but still display all necessary info.

I'm assuming there's no JSON endpoint on the game server we can hit for a spells table—would we be OK scraping the wiki?

We'll be fine scrapping the wiki for most things, I've done it for other stuff

If there are data issues, you can just ping me and I'll edit whatever is wrong, I should still have authorship privileges there.

ondreian commented 4 years ago

This is mostly done, maybe needs a couple followups if there is missing data.