Skip to content

Various changes#130

Open
carasmo wants to merge 2 commits intokraftbj:developfrom
carasmo:develop
Open

Various changes#130
carasmo wants to merge 2 commits intokraftbj:developfrom
carasmo:develop

Conversation

@carasmo
Copy link
Copy Markdown

@carasmo carasmo commented Jun 17, 2017

  • Valid html when used more than once on a page
  • made JS friendly variable
  • Changed all screenread classes to screen-reader-text used in all Genesis themes
  • Removed #button on submit forms and added class button
  • removed inline css for screenread since the class is now screen-reader-text

carasmo added 2 commits June 16, 2017 18:23
In the ``BJGK_Genesis_eNews_Extended`` class, the label class was changed to screen-reader-text that comes with all Genesis child themes. Plus it's the class WordPress introduced in 2009 https://www.google.com/search?q=wordpress+screen-reader-text&ie=utf-8&oe=utf-8
- Valid html when used more than once on a page
   -- made JS friendly variable
- Changed all ``screenread`` classes to ``screen-reader-text`` used in all Genesis themes
- Removed #button on submit forms and added class ``button``
@carasmo
Copy link
Copy Markdown
Author

carasmo commented Jun 17, 2017

I tested this out, and it works fine with MailChimp. The code is valid html when I use this in a widget twice on the page. The only time it is odd and this is something I would mark as "not fixing" is using inside a widget created with a page builder, it doesn't error and works, but the instance is #subbox1_enews_ext_REPLACE_TO_ID (for example) instead of #subbox1_enews_ext_4 (the number changes each instance). The way around that, which I will write up a post for, is to use the shortcode any widget plugin, create the instance in the widgets panel, and use a shortcode inside the page builder. Very easy work around.

Repository owner deleted a comment from carasmo Jun 18, 2017
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.

1 participant