ctubbsii commented on code in PR #398:
URL: https://github.com/apache/accumulo-website/pull/398#discussion_r1309156146
##########
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:
Why move away from the glyphicons halflings? Are those no longer allowed in
newer bootstrap? Have you considered using [the bootstrap icons
stuff](https://icons.getbootstrap.com/#install) rather than font-awesome?
--
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
