tag:help.tenderapp.com,2008-11-12:/discussions/problems/212-site-link-should-have-target_topTender: Discussion 2018-10-18T06:25:52Ztag:help.tenderapp.com,2008-11-12:Comment/340692009-01-14T21:41:18Z2009-06-20T20:17:00ZSite link should have target=_top<div><p>Hmm, I'd really rather not do this as I don't really want to
assume people are using frames. However, given the fact that you
are using tender within a frame, do you even want this link inside
tender? If not, I'd recommend putting this bit in your custom CSS
to remove it:</p>
<pre>
<code>#gnav .visit_site{ display:none; }</code>
</pre>
<p>I'll be pushing on to see how long it'll take to get HTML
headers and footers in safely.</p></div>Kyle Neath (GitHub Staff) tag:help.tenderapp.com,2008-11-12:Comment/340692009-01-15T16:14:36Z2009-01-15T16:18:07ZSite link should have target=_top<div><p>That is a great suggestion, but the css you sent doesn't seem to
have an effect. The #gnav .visit_site {display} attribute is
overridden by "#superheader ul#gnav li" according to Safari's web
inspector. I'm awful with css and haven't found an easy override
for that. Would you mind providing a bit of css that overrides
that, if possible?</p>
<p>Update: Oops, nevermind I found the right css:
<code>#superheader ul#gnav li.visit_site { display:none;
}</code></p></div>Sean Reillytag:help.tenderapp.com,2008-11-12:Comment/340692009-01-15T20:10:29Z2009-01-15T20:10:29ZSite link should have target=_top<div><p>Cool, glad it worked out for you!</p></div>Kyle Neath (GitHub Staff) tag:help.tenderapp.com,2008-11-12:Comment/340692009-03-11T16:25:14Z2009-03-11T16:25:14ZSite link should have target=_top<div><p>The CSS mentioned above no longer works. Is there some
new/reliable way to hide that link via CSS?</p>
<p>Adding the <em>top target would be even better. I understand not
wanting to assume the site is loaded in a frame, but including
target=</em>top should work just fine whether using frames or not,
and I don't see any way that this could be a security risk.</p>
<p>(just signed up for the mid-range subscription)</p></div>Sean Reillytag:help.tenderapp.com,2008-11-12:Comment/340692009-03-12T06:33:22Z2009-03-12T06:33:22ZSite link should have target=_top<div><p>Sorry about that — we had to change the ul#gnav to
ul.gnav, edit those lines and it should be good to go. If I include
target=top, it'll actually open in a new window by default (right
now it's a regular link). So while it'll work without frames, it'll
make the link act differently.</p></div>Kyle Neath (GitHub Staff) tag:help.tenderapp.com,2008-11-12:Comment/340692009-03-12T13:41:43Z2009-03-12T13:42:53ZSite link should have target=_top<div><p>Thanks for your attention, but the CSS you provided still
doesn't work (see http://moneydance.com/tender)</p>
<p>I should also clarify that the code is
<code>target="_top"</code> (the underscore before 'top' was sucked
out by markdown before). If the link is not contained within a
frame then it does no harm as '_top' is a special reference to the
top-level frame (if any) of the browser window. For more
information see section 6.16 of <a href=
"http://www.w3.org/TR/html401/types.html#type-frame-target">the
HTML spec</a></p>
<p>If the page is not shown in frames, the '_top' target will
<em>not</em> open a new window/tab/frame when the link is clicked.
Please consider adding it.</p></div>Sean Reilly