DomGarguilo commented on code in PR #398:
URL: https://github.com/apache/accumulo-website/pull/398#discussion_r1309170727
##########
pages/contact-us.md:
##########
@@ -45,10 +45,10 @@ follow discussions as they happen.
{: .table }
| Name | Description | Read
| Follow | Post |
|-------------------|--------------------------------------------------|------|--------|------|
-| **user** | General user questions, help, and announcements |
[<span class="glyphicon glyphicon-search"/> Archive][U_A] | [<span
class="glyphicon glyphicon-plus"/> Subscribe][U_SU] [<span class="glyphicon
glyphicon-remove"/> Unsubscribe][U_UN] | [<span class="glyphicon
glyphicon-envelope"/> Post][U_P] |
-| **dev** | Contributor discussions and development activity |
[<span class="glyphicon glyphicon-search"/> Archive][D_A] | [<span
class="glyphicon glyphicon-plus"/> Subscribe][D_SU] [<span class="glyphicon
glyphicon-remove"/> Unsubscribe][D_UN] | [<span class="glyphicon
glyphicon-envelope"/> Post][D_P] |
-| **commits** | Code changes |
[<span class="glyphicon glyphicon-search"/> Archive][C_A] | [<span
class="glyphicon glyphicon-plus"/> Subscribe][C_SU] [<span class="glyphicon
glyphicon-remove"/> Unsubscribe][C_UN] | |
-| **notifications** | Automated notifications (GitHub, etc.) |
[<span class="glyphicon glyphicon-search"/> Archive][N_A] | [<span
class="glyphicon glyphicon-plus"/> Subscribe][N_SU] [<span class="glyphicon
glyphicon-remove"/> Unsubscribe][N_UN] | |
+| **user** | General user questions, help, and announcements | [<i
class="fa-solid fa-magnifying-glass"></i> Archive][U_A] | [<i class="fa-solid
fa-plus"></i> Subscribe][U_SU] [<i class="fa-solid fa-xmark"></i>
Unsubscribe][U_UN] | [<i class="fa-solid fa-envelope"></i> Post][U_P] |
+| **dev** | Contributor discussions and development activity | [<i
class="fa-solid fa-magnifying-glass"></i> Archive][D_A] | [<i class="fa-solid
fa-plus"></i> Subscribe][D_SU] [<i class="fa-solid fa-xmark"></i>
Unsubscribe][D_UN] | [<i class="fa-solid fa-envelope"></i> Post][D_P] |
Review Comment:
Yea glyphicons wasn't working. Switching to bootstrap icons would require an
extra import but could be worth it potentially if we wanted to replace
font-awesome with bootstrap icons but that should probably happen in a separate
PR
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
