Add class to hide elements if guest visitor [#100512234]

[dougmartin]

NOTE: This PR has been changed to a more generic role based class system instead of a guest visitor focused change. The body tag now has a 'xxx-visitor' class assigned per role, eg 'guest-visitor' and 'author-visitor'. There are additional classes created in a loop in the web.scss file to:

1. Hide an element from all visitors: body .hide-from-all-visitors
2. Show an element for only a specific role: body.guest-visitor .show-to-guest-visitor
3. Show an element inline for only a specific role: body.guest-visitor .show-inline-to-guest-visitor
4. Hide an element for only a specific role: body.guest-visitor .hide-from-guest-visitor

To show the "Create Activity" button only to authors the ITSI project landing page content needs to have the linked wrapped in an element with the "hide-from-all-visitors show-to-author-visitor" classes. I've added it to https://itsi-master-staging.concord.org/itsi already.

<p class="hide-from-all-visitors show-to-author-visitor">
  <a href="https://itsi-master-staging.concord.org/eresources/433/copy" class="button pie" target="_blank">Create ITSI Activity</a>
</p>

[scytacki]

Now that I see this, it should be hide-if-not-author instead. Or perhaps
less negative show-only-to-authors

from my phone
On Aug 11, 2015 5:43 AM, "Doug Martin" notifications@github.com wrote:

NOTE: To hide the "Create Activity" button the ITSI project landing page
content needs to have the linked wrapped in an element with the
"hide-if-guest-visitor" class. I've added it to
https://itsi-master-staging.concord.org/itsi already.

Create ITSI Activity

---

You can view, comment on, or merge this pull request online at:

#140
Commit Summary

• Add class to hide elements if guest visitor [#100512234]

File Changes

• M app/assets/stylesheets/web/app.scss
  https://github.com/concord-consortium/rigse/pull/140/files#diff-0
  (6)
• M app/views/layouts/application.html.haml
  https://github.com/concord-consortium/rigse/pull/140/files#diff-1
  (2)

Patch Links:

• https://github.com/concord-consortium/rigse/pull/140.patch
• https://github.com/concord-consortium/rigse/pull/140.diff

—
Reply to this email directly or view it on GitHub
#140.

[dougmartin]

@scytacki I've made the PR a more generic role based class system. See the PR description for updated info.

[scytacki]

Looks good to me. 👍