Skip to content
This repository was archived by the owner on Jun 20, 2018. It is now read-only.

Cannot add conditions in Nexus#98

Open
arnaudlimbourg wants to merge 1 commit intodisqus:masterfrom
arnaudlimbourg:patch-1
Open

Cannot add conditions in Nexus#98
arnaudlimbourg wants to merge 1 commit intodisqus:masterfrom
arnaudlimbourg:patch-1

Conversation

@arnaudlimbourg
Copy link

When the switch name has spaces it is not possible to add conditions to make the switch selective. When adding underscores it works properly though.

I have not investigated the nexus code base but I figured updating the documentation made sense.

When the switch name has spaces it is not possible to add conditions to make the switch selective. When adding underscores it works properly though.

I have not investigated the nexus code base but I figured updating the documentation made sense.
adamchainz pushed a commit to adamchainz/gargoyle that referenced this pull request Feb 12, 2016
Fixes the issue reported in disqus#98, thanks @arnaudlimbourg.

Turns out that the rows were being assigned HTML ID's, which are not
allowed to feature spaces, however this is unneccesary since we only use
the `data-` attributes to reference the HTML.
@adamchainz
Copy link
Contributor

Fixed using spaces in Switch keys on the YPlan fork in YPlan/gargoyle#38 , thanks for the report.

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.

2 participants