You are viewing a single comment's thread from:

RE: The Steem Curation Extension now shows available voting power

Update 2: Apparently, when you log out and back in, none of the events that I was handling take place. With Claude's help, I set up a mutation observer to add the click handler when the dropdown menu gets created, and I also added a check to eliminate the errors when the dropdown menu doesn't exist. This branch is an updated version of yours: https://github.com/remlaps/Steem-Curation-Extension/tree/fix4NoDropdown

Sorry if I'm not handling any of the git stuff correctly, I've never worked on an actively shared repo in git, so I'm figuring it out as I go.

Sort:  

You were much quicker than I expected :-D My branch wasn't even finished yet because I hadn't tested it properly either. In addition, the code was still very wild. But I'm glad you've already been able to fix something.

This branch is an updated version of yours:

I've had a look at it and am already very excited. The code doesn't look so wild anymore. :-)
Working together on a branch is yet another new challenge. I don't know if we can use comments to discuss the code directly.
Unfortunately it's too late today (again), but I'm still happy to test the code soon.

You were much quicker than I expected :-D My branch wasn't even finished yet because I hadn't tested it properly either. In addition, the code was still very wild. But I'm glad you've already been able to fix something.

Coincidentally, I had the morning off today, and I was (overly?) eager to figure out what was going on with that 1st click not displaying.

I've had a look at it and am already very excited. The code doesn't look so wild anymore. :-)

Yeah, after two years, I didn't remember much about how it worked, so I tried to prepare for next time. Although, this exercise really gave me some ideas about other possibilities, so maybe it won't be two years again.

Working together on a branch is yet another new challenge.

Yeah, sorry about that, I was not sure about the correct github mechanics for the situation, but the way I did it seemed less than ideal. FWIW, I don't expect to be working on it again in the next few days, so test away if you find time. 😉 Once you're satisfied with the current iteration, I think we can update the version date and merge it into the main branch. There's still one scenario (that I'm aware of) where the highlighting doesn't get painted on until the user scrolls or clicks, but that's been with us since the beginning. Not a result of these recent changes.