Skip to content
This repository has been archived by the owner on Jan 16, 2024. It is now read-only.

Remove is_element_empty() #95

Merged
merged 1 commit into from
May 22, 2015

Conversation

kalenjohnson
Copy link
Contributor

Move is_element_empty into closure for Nav Walker. Especially since is_element_empty was lying

@kalenjohnson kalenjohnson force-pushed the fix_is_element_empty branch from 22728a1 to 0ad023b Compare May 21, 2015 23:56
swalkinshaw added a commit that referenced this pull request May 22, 2015
@swalkinshaw swalkinshaw merged commit fdfe262 into roots:master May 22, 2015
@QWp6t
Copy link
Member

QWp6t commented Sep 24, 2015

Could this work as an alternative?

$classes = array_map('trim', $classes);
$classes = array_filter($classes);

I'm on my phone and the app I'm using doesn't let me see the code while I'm writing this message, but I might have inverted the logic. If so, array_diff should cover it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants