Add checkmark icon instead of (Answered) to questions display in pages #1624
Labels
design
issue requires more design work and discussion (i.e. mockups and sketches)
enhancement
explains that the issue is to improve upon one of our existing features
first-timers-only
They need to be well-formatted using the First-timers_Issue_Template.
help wanted
requires help by anyone willing to contribute
HTML
Hi, this is a first-timers-only issue. This means we've worked to make it more legible to folks who either haven't contributed to our codebase before, or even folks who haven't contributed to open source before.
If that's you, we're interested in helping you take the first step and can answer questions and help you out as you do. Note that we're especially interested in contributions from people from groups underrepresented in free and open source software!
If you have contributed before, consider leaving this one for someone new, and looking through our general help wanted issues. Thanks!
The problem
Questions grids, generated by the inline shortcode
[questions:TAGNAME]
(documented here) now show questions that have been answered with a nice prefix,(Answered)
-- which is helpful to tell which have been answered! But it takes up a lot of space:https://publiclab.org/wiki/harvey#Questions
Solution
This display is set on these lines of code in the grids templates: https://github.com/jywarren/plots2/blob/double-inline-shortcodes/app/views/grids/_notes.html.erb#L37
What we should do is change this:
so that instead of
<b>(Answered)</b>
it says<i class="fa fa-"></i>
(note the trailing space!)Extra credit!
I found the checkbox icon (http://fontawesome.io/icon/check-square/) from http://fontawesome.io/icons/, which accounts for the
fa fa-___
classes. We have FontAwesome installed, so that makes this easier!There are other checkbox icons there, so if you're opinionated about it, you could try another. Or turn it green with
<i style="color:#0c8;" ...
.Steps to Fix
Please email the developers list (see https://publiclab.org/wiki/developers) or go to the chatroom if you have questions, and take a look at our first-timers landing page for more information!
The text was updated successfully, but these errors were encountered: