You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As per adaptlearning/adapt-contrib-vanilla#443, the default style for Notify icon buttons is consistent across all plugins however we've duplicated the styling code across the various plugin LESS files in Vanilla and Hotgrid.
Suggest applying .notify__btn-icon to .hotgrid-popup__controls and .hotgrid-popup__close so styles inherit from Vanilla notify.less. Duplicate styles can then be removed from the LESS.
This means the default style is set in one place but we still have plugin specific classes to target should you want to style these different - although there's good reason for keeping UI button styling consistent.
The text was updated successfully, but these errors were encountered:
Subject of the enhancement
As per adaptlearning/adapt-contrib-vanilla#443, the default style for Notify icon buttons is consistent across all plugins however we've duplicated the styling code across the various plugin LESS files in Vanilla and Hotgrid.
Suggest applying
.notify__btn-icon
to.hotgrid-popup__controls
and.hotgrid-popup__close
so styles inherit from Vanilla notify.less. Duplicate styles can then be removed from the LESS.This means the default style is set in one place but we still have plugin specific classes to target should you want to style these different - although there's good reason for keeping UI button styling consistent.
The text was updated successfully, but these errors were encountered: