17

When I try to edit my comment, the textbox shows [object Object] instead of the previous comment text. First time I've seen this. Xubuntu 16.04, Chrome 63.

Image of regular comment:

screenshot 1

Image of comment box after I click it:

screenshot 2

If you can't see the image, it should show:

What does this do? (I don't know Perl)

but it shows:

[object Object]

Link to post.

@SebastianSimon pointed out this duplicate on Meta.SE, so it looks like it started very recently.

8

1 Answer 1

10

Cross post from MSE

This is my fault, the fix is live now for MSE, building on the network.

In working on Keyboard-only users cannot flag comments in the new comment flag dialog, I needed access to the event that showed the flagging dialog, so I updated the comment UI hooks to pass in the event as the first parameter.

This was fine fo the various hooks, upvote, flag, etc, except for edit, because the edit hook already had a parameter defined textOverride. So the event that was passed in was treated as a text override and replaced the edit text.

6
  • and the new focus state on the comment icon is intentionnal or a side effect of something? Commented Aug 17, 2018 at 19:58
  • The fix is live for SO also. No longer getting [Object object] here.
    – Bhargav Rao Mod
    Commented Aug 17, 2018 at 19:59
  • 15
    @TemaniAfif If you're talking about upvote/flag, it's intentional. They're now buttons and they're inheriting styles from the .s-btn class. It was one option to make them keyboard/screen reader accessible.
    – Brian Nickel Staff
    Commented Aug 17, 2018 at 20:48
  • 4
    @BrianNickel Can we get the outlines to go away after the relevant action's been taken? As is when using a mouse they're invisible until after you click the button and don't go away until you click something else. Commented Aug 21, 2018 at 0:52
  • @Draco18s I have a change in review right now that would reduce the highlighting considerably and prevent the icon color from turning that dark blue. Turns out there was a mess of poorly interacting styles that needed to be cleaned up. As far as making the highlight go away completely for mouse users, it looks like that's non-trivial and surrounded with browser quirks.
    – Brian Nickel Staff
    Commented Aug 21, 2018 at 21:13
  • @BrianNickel Thanks for the reply. I'm disappointed, but appreciate the response. Commented Aug 22, 2018 at 17:01

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .