publiclab / PublicLab.Editor

A general purpose, JS/Bootstrap UI framework for rich text posting. An author-friendly, minimal, mobile/desktop interface for creating blog-like content, designed for PublicLab.org

Home Page:https://publiclab.github.io/PublicLab.Editor/examples/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Column subtract button in table popover has no style

NARUDESIGNS opened this issue · comments

The - button for columns has no style, making it difficult to tell whether or not it is a button.
It should match the style of the other buttons.
See image for you reference:
Screenshot 2021-11-04 at 22 48 05

Hi @jywarren, trust you're good?
I remember you mentioned where the HTML file for the Public Lab Editor is but I can't seem to remember, can you please remind me?
I found the cause of this issue, it's in the HTML code. The class name was misspelt.
cc @TildaDares

Thank you so much @TildaDares
However, I checked that file and many others but couldn't find the element I'm looking for (decCol). It has a class name of btn-outline-secondaryt. The extra "t" makes the element unable to access the style. Here's an image to show you the element I'm looking for:

Screenshot 2021-11-13 at 11 06 41

Any help?

Hi! Great find on that typo. The line of code is here -- it's a "submodule" of the rich text module!

builder += '<a id="decCols" class="btn btn-sm btn-outline-secondaryt"><i class="fa fa-minus"></i></a> <span id="tableCols">3</span> <a id="incCols" class="btn btn-sm btn-outline-secondary"><i class="fa fa-plus"></i></a>';

@jywarren Yea thanks, I think my PR captured that as well. Please have a look at the changes in the code here
NARUDESIGNS@d7e2a0a

Screenshot 2021-11-14 at 20 27 35

Ah, actually @NARUDESIGNS would you mind opening a PR for your changes? It's easier to keep all our reviews in one place then. Thank you!

@jywarren I opened a PR for it here - #758
@TildaDares was kind to review and asked for some changes which I've also completed. I'm waiting for my PRs to be merged. This is one of them, there's another (this one #757 which fixes issue #755)

Was this now resolved? Just checking - we can close it if so!

Was this now resolved? Just checking - we can close it if so!

Yes, you just merged it #758