Skip to content
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

Fix filter_vertical rendering #358

Closed
wants to merge 3 commits into from

Conversation

boydjj
Copy link

@boydjj boydjj commented Apr 17, 2015

I updated the LESS using the CSS submitted by @szymanskilukasz in #322 so that filter_vertical fields render properly. This is useful for M2M fields with long __str__/__unicode__ values. I also confirmed that filter_horizontal fields still render correctly. Screenshots below.

I also documented the version of less required (as mentioned in the IRC channel) and cleaned up an apparent typo mishap. Hope that's all right!

Before screenshot:
change_experiment___experiment_engine_admin

After screenshot:
change_experiment___experiment_engine_admin

@boydjj
Copy link
Author

boydjj commented Apr 17, 2015

Just a note: this failing build looks like an issue w/ Travis integration and has happened on other branches.

@SalahAdDin
Copy link

👍 It's seen very good.

@darklow
Copy link
Owner

darklow commented Apr 22, 2015

I will merge in your .less code manually, because i also updated LESS version and those two last commits for PR are redundant.
Thank you for your PR.

@darklow darklow closed this Apr 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants