UI bug where both approval and removal buttons are visible

themotte / rDrama

This code runs https://www.themotte.org. Forked from https://github.com/Aevann1/rDrama

GNU Affero General Public License v3.0

25 stars 31 forks source link

UI bug where both approval and removal buttons are visible #600

Closed zorbathut closed 1 year ago

zorbathut commented 1 year ago

From https://github.com/themotte/rDrama/pull/596:

There's a UI bug where both approval and removal buttons are visible at once, even on the happy path. Fix: s/d-md-inline-block/d-md-block/g on files/assets/js/comments_admin.js. Think the template changed at some point which wasn't reflected in JS (and pre-dates this change).

justcool393 commented 1 year ago

was this fixed by #637? since we're not replacing button inner html anymore (just hiding (unhiding) the same (opposite) buttons), this should be fixed

zorbathut commented 1 year ago

Probably, yeah. I'll close it.