Project

General

Profile

Actions

Feature #10712

closed

Open Site Template demo links in new window?

Added by Boone Gorges almost 6 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
-
Category name:
Blogs (BuddyPress)
Target version:
Start date:
2018-11-16
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Related issues

Related to CUNY Academic Commons - Feature #9943: Site templatesResolvedBoone Gorges2018-06-15

Actions
Actions #1

Updated by Boone Gorges almost 6 years ago

Actions #2

Updated by Boone Gorges almost 6 years ago

  • Category name set to Blogs (BuddyPress)
  • Target version set to Future release

From https://redmine.gc.cuny.edu/issues/9943#note-30:

did we discuss opening links to Demo sites via the Site Template picker in a new window? Might make sense in the site creation flow to do that.

Best practices (usability, accessibility) are generally strongly against new-window links. If the user wants to open in a new window, it's possible using browser tools; if we set target="_blank", there's no way to disable the behavior. That said, I understand the reasoning behind it, and we can do it if necessary.

Another possible strategy is to add text to the effect of "(Right-click to open in new tab)" or something like that next to the demo links, though the instructions would need to be sensitive to browser/OS.

Actions #3

Updated by Matt Gold almost 6 years ago

I agree that we should not open links in new windows. I was a big fan of target="_blank" in the early days of the web, but I have been convinced by usability reports and other pieces that this is no longer in line with best practices of web design.

some resources/rationales for this:

https://medium.com/the-metric/links-should-open-in-the-same-window-447da3ae59ba

https://blog.tbhcreative.com/2015/07/pros-and-cons-of-opening-links-in-new-window.html

https://accessibility.oit.ncsu.edu/it-accessibility-at-nc-state/developers/accessibility-handbook/mouse-and-keyboard-events/links/links-and-new-windows/

Actions #4

Updated by Luke Waltzer almost 6 years ago

Yes— I’m aware of accessibility issues and best practices here... but are we asking people to enter information (site URL/name/license/viability) and then subsequently inviting them to navigate away from that form to explorr the demo sites before submitting site request? Is the data they’ve entered being stored in the browser for them to click back to? What if they click around the demo site for a while?

Tried to test for myself, but am on mobile and links to demo sites are not visible there.

Actions #5

Updated by Boone Gorges almost 6 years ago

We could add one of those "you are about to leave the page" JS warnings if someone has entered something into the form.

Also, if there are mobile styling issues, we should probably handle them as a separate thread.

Actions #6

Updated by Luke Waltzer almost 6 years ago

Think js warning plus instruction on how to open in new window would do the trick

Actions #7

Updated by Boone Gorges almost 6 years ago

  • Target version changed from Future release to 1.14.1

Thanks. I'll see what I can do for the next release.

Actions #8

Updated by Boone Gorges almost 6 years ago

  • Target version changed from 1.14.1 to 1.14.2
Actions #9

Updated by Sonja Leix almost 6 years ago

I understand Luke's concerns. I think the JS warning will likely work for now.

For a later and more user friendly version of this where the user isn't confronted with a warning at all, maybe we could implement what we often see on WordPress theme demo sites like this one: https://my.studiopress.com/themes/essence/#demo-full. See bar at the bottom that takes you back to where you were. It seems to not navigate you way from the site at all to display the demo, but rather loads the demo in a frame over the page. (text closing the demo, and then click "Live Demo" under the price tag again).

Actions #10

Updated by Boone Gorges almost 6 years ago

  • Status changed from New to Resolved

Thanks, all. I've made the following changes:

1. Added the "you are about to leave" messages. Most modern browsers do not support custom messages, so in most cases you'll see whatever text the browser generally shows you in such cases. https://github.com/cuny-academic-commons/cac-site-templates/commit/6acd4c635391d2624e1e8fc40a2c611b12cb9861

2. Added help text for opening in a new window. https://i.imgur.com/KRFAup6.png https://github.com/cuny-academic-commons/cac-site-templates/commit/b15447ee3c421bb139ddfe1c65ee4775c64db62e

This will be live after the next release.

Actions

Also available in: Atom PDF