Kalliope wrote:Edit edit: Are your personal notes going to go back on the pages at all? Or are the links to wowhead, etc. going to be it?
You'll find some short comments in the notes section, but I removed some old ones that were out of date and shortened some of the others. I need to do a pass through the data and add a lot of notes yet.
Moonlost wrote:I'm particuarly fond of the "Non-*pet family*s With Similar Looks" that are on relevent pages. I'd like to see the ghost saber added to the spirit beast page, under "Non-Spirit Beasts With Similar Looks".
Good idea! I also added the other ghostly pets to the Spirit Beast page as similar looks, and things like the slime. Let me know if you see any others that should be added!
FuzzyDolly wrote: As I said before it looks awesome, however... when you're looking at an animals page, there is no longer the thottbot link to get to a map of the pet's location. I used that feature almost every time I checked Petopia. Any chance on putting it back or even just linking to a map directly?
I wanted to shorten the text links ("Thottbot - Allakhazam - Wowhead") down into something shorter. I've used just initials before ("T-A-W") but I have trouble clicking on the tiny little links. I figured the icons for each site were a good compromise, but if they don't stand out enough then that doesn't quite work. Do you think it would help if they were in their own column in the table?
Sarayana wrote:A slight bug: When I was browsing warp stalkers (don't know about any other families) the pop-up didn't work properly. It didn't pop up for the three at the bottom until I had hovered over the top left one, and once it did, it just was the same pop-up as that top left one. The text I mean.
No repro. *grin*
Okay, I actually just fixed this up real quick.
Safrienaer wrote:Also it seems that the Nether Rays have no npcs shown at all.

No idea if you just haven't gotten to making them yet, though.
Damn you, "Count" Ungula!
But that bug should also be fixed now.
Explanation for these bugs, if you care: I was originally setting the text for the pretty tooltips in the HTML itself via the title attribute (and JavaScript that turns that into a tooltip). But that means that if you have JavaScript turned off, you still see the normal 'title' hover text -- which is now a huge block of HTML code.
So yesterday I switched to setting the pretty tooltip text programmatically in the JavaScript itself. That works fine, except for two things:
- If there is no tooltip (as for untameable beasts), I wasn't setting the tooltip to an empty string ... and so it just used the last tooltip you saw. (This didn't matter before because the title attribute would have been "".) I fixed this by removing any references to empty tooltips, which I shoudl have done in the first place.
- When the tooltips were set in the HTML, they needed to have any internal HTML turned into non-HTML first. So < became < for example. The code was turned back into real HTML before being shown as a tooltip. When you set the tooltip text programmatically, though, you don't need to do that. So I didn't. But that still leaves one unsafe character - a double quote (") - and as it happens there is a beast with a double quote in his name: "Count" Ungula. So he broke the Nether Ray page. Now I explicitly escape double quotes in the tooltip text.