-
Notifications
You must be signed in to change notification settings - Fork 9
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
Release 0.2.1 #129
Release 0.2.1 #129
Conversation
@valerymelou sorry, for punking you. Also updated the no_request tests, to actually test proberly. |
Also test coverage has finally reached 100% 🥇 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #129 +/- ##
===========================================
+ Coverage 96.90% 100.00% +3.09%
===========================================
Files 6 6
Lines 97 106 +9
Branches 12 14 +2
===========================================
+ Hits 94 106 +12
+ Misses 1 0 -1
+ Partials 2 0 -2 ☔ View full report in Codecov by Sentry. |
): | ||
""" | ||
Renders the given CSS class if the request path matches the path of the view. | ||
:param context: The context where the tag was called. Used to access the request object. | ||
:param viewnames: The name of the view or views separated by || (include namespaces if any). | ||
:param css_class: The CSS class to render. | ||
:param inactive_class: The CSS class to render if the views is not active. | ||
:param css_inactive_class: The CSS class to render if the views is not active. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a small grammar error here: if the view
Make use of the inactive class and properly document it.
Also add new setting to settings to preset inactive class.