-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MS Team icon text is missing white space when plugins are icons are in collapsed list #23
Comments
I wonder if there has been any changes in the CSS. What I think is happening here is that SVGs are being treated different as font-icon in terms of spacing. Not 100% sure if where should we fix those. |
In order to have proper styling, the elements must have the "icon" class. We can solve this in two different ways:
Thoughts? |
Option 2. seems like the right way to me. Plugins shouldn't rely on some CSS class that the webapp uses. |
Agree with @hanzei that we would ideally not be brittle in depending on CSS classes, but I admit that ship has probably long since sailed. Can we do "both" somehow, by having the plugin conditionally add the |
If we were to do both, we would have to change "all the plugins". Is it something we want to do? For short term I will just add the icon class to MSTeams, and if we agree on a long term solution, we can file the needed tickets both in the main product and each of the plugins. |
@larkox, could we only change the plugins that "use the icon class already", as proposed in Option 2? Effectively allowing those plugins to continue to support older webapp versions than the one in which this class fix will ship? |
When MS teams calling is enabled along with several others, there is missing white space between the plugin icon and title
The text was updated successfully, but these errors were encountered: