-
Notifications
You must be signed in to change notification settings - Fork 888
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
This shows the correct view when a class and attr
is involved.
#2687
Conversation
Before this change:
and after:
|
Nice. For completeness, should either of these be updated?
Also a change log entry? |
Sorry, my bad aim on clicking buttons. |
@stevepiercy Nope, the docs are accurate, there was just this tiny little bug when you used a class + |
@sontek LGTM. Should this bug fix be backported to 1.7 and 1.6, along with change log entries? |
@stevepiercy Yeah, I think it'd be a great candidate for a backport |
LGTM. This feature is fine to be backported if you want to submit the PRs and changelog entries for them. |
I'll do backports tomorrow. |
Backport #2687 to 1.7-branch
No description provided.