VPF::Добавить в Избранное - Форум программистов

Wolf1994

Дата 13.2.2008, 10:48 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1701
Регистрация: 5.10.2004

Репутация: 4
Всего: 29

Код


function fav_add()
{
 var url="http://"+document.domain;
 var title=document.title;
 if(window.sidebar)
  window.sidebar.addPanel(title,url,"");
 else if(window.opera&&window.print)
 {
  var mbm=document.createElement('A');
  mbm.setAttribute('rel','sidebar');
  mbm.setAttribute('href',url);
  mbm.setAttribute('title',title);
  mbm.click();
 }
 else if(document.all)
  window.external.AddFavorite(url,title);
}

При вызове скрипта страница обновляется. Что неверно в этом коде? Opera: 9.25, система: Windows XP SP2.

Ghirik

Дата 13.2.2008, 11:55 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 919
Регистрация: 26.3.2007

Репутация: 13
Всего: 20

Может лучше такую функцию применить?

Код


function bookmark() {
    if (window.sidebar) {
        window.sidebar.addPanel(document.title, window.location,"");
    } else if (document.all) {
        window.external.AddFavorite(window.location, document.title);
    }
}

--------------------

Отдадим всё клиенту, пускай его машина мучается...

Wolf1994

Дата 13.2.2008, 12:07 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1701
Регистрация: 5.10.2004

Репутация: 4
Всего: 29

Ghirik, попробовал. FF & IE методы в Opera не работают.

Ghirik

Дата 13.2.2008, 12:30 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 919
Регистрация: 26.3.2007

Репутация: 13
Всего: 20

Ага... Есть такое.... А я по привычке в FF и IE проверил, а до Оперы не добрался... Не я писал то... где то выдрал... Сейчас, попробую разобраться.

--------------------

Отдадим всё клиенту, пускай его машина мучается...

KuZyagroup

Дата 13.2.2008, 12:40 (ссылка)

(нет голосов)

Загрузка ...

magento solution

Профиль
Группа: Участник
Сообщений: 283
Регистрация: 16.4.2007
Где: Kiev

Репутация: 1
Всего: 4

подозреваю что проблема в click
например вот этот простенький код вызовет ошибку

Код


<a id="iA" rel="sidebar" href="trulala">to bookmark</a>
<script>
(function doit(){
    document.getElementById('iA').click();
})()
</script>

ошибку click is not a function

Wolf1994

Дата 13.2.2008, 12:56 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1701
Регистрация: 5.10.2004

Репутация: 4
Всего: 29

KuZyagroup, в Opera эта функция ошибки не вызывает, но и не работает. Пример кода приведённый мной я встречал на нексольких сайтах, как рабочий для Opera 7+. Также, насколько я помню, он работал в более ранних версиях этого браузера.

KuZyagroup

Дата 13.2.2008, 13:17 (ссылка)

(нет голосов)

Загрузка ...

magento solution

Профиль
Группа: Участник
Сообщений: 283
Регистрация: 16.4.2007
Где: Kiev

Репутация: 1
Всего: 4

В таком случае можно предположить что в опере решили не этично делать носильное предложение добавить страницу в закладку(особенно с учётом того что его можно зациклить;) и этим положить браузер), а редирект и без клика делается.
С другой стороны в DOM Snapshot чётко видно что у оперы есть Click для A. Тогда не понятно почему мой код не работает

Это сообщение отредактировал(а) KuZyagroup - 13.2.2008, 13:18

Wolf1994

Дата 13.2.2008, 13:29 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1701
Регистрация: 5.10.2004

Репутация: 4
Всего: 29

А можно ли сгенерировать события нажатия на клавиши: "Ctrl+D"?

Ghirik

Дата 18.2.2008, 09:33 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 919
Регистрация: 26.3.2007

Репутация: 13
Всего: 20

Вот нашел сайт где работает добавление в избранное и в Опере. Хот и с ошибками, но окно добавления выскакивает.

Ни как не могу выдернуть саму функцию, в атрибуте onclick указано window.external.AddFavorite (top.location.href,top.document.title); Но эта штука не работает... Видимо где то имеется переназначение.

Может кто разберется?

--------------------

Отдадим всё клиенту, пускай его машина мучается...

ksnk

Дата 18.2.2008, 11:17 (ссылка)

(нет голосов)

Загрузка ...

прохожий

Профиль
Группа: Комодератор
Сообщений: 6855
Регистрация: 13.4.2007
Где: СПб

Репутация: 84
Всего: 386

Дык!

Код


<a href="#gohere"  rel="sidebar" name="gohere" >aaaaaaaaa</a>

--------------------

Человеку свойственно ошибаться, программисту свойственно ошибаться профессионально ! user posted image

DE20

Дата 18.2.2008, 11:30 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 30
Регистрация: 9.2.2008

Репутация: 1
Всего: 1

Ghirik,
rel="sidebar" необходим

<a href="#" onclick="window.external.AddFavorite (top.location.href,top.document.title);" rel="sidebar" > Добавить в избранное</a>

Wolf1994

Дата 18.2.2008, 12:16 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1701
Регистрация: 5.10.2004

Репутация: 4
Всего: 29

Всем спасибо. Вопрос решён.

Ghirik

Дата 18.2.2008, 13:11 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 919
Регистрация: 26.3.2007

Репутация: 13
Всего: 20

Да, работает везде... Но Опера находит ошибку:

Цитата

JavaScript - ....
Event thread: click
Error:
name: TypeError
message: Statement on line 1: Could not convert undefined or null to object
Backtrace:
  Line 1 of  script
    window.external.AddFavorite(top.location.href, top.document.title);
return false;
  At unknown location
    [statement source code not available]

В чем прикол?

Это сообщение отредактировал(а) Ghirik - 18.2.2008, 13:12

--------------------

Отдадим всё клиенту, пускай его машина мучается...

ksnk

Дата 18.2.2008, 13:39 (ссылка)

(нет голосов)

Загрузка ...

прохожий

Профиль
Группа: Комодератор
Сообщений: 6855
Регистрация: 13.4.2007
Где: СПб

Репутация: 84
Всего: 386

Ghirik,

Код


<a href="#" onclick="window.external && window.external.AddFavorite (top.location.href,top.document.title);" rel="sidebar"  > Добавить в избранное</a>

вот так не будет ошибки в Опере

--------------------

Человеку свойственно ошибаться, программисту свойственно ошибаться профессионально ! user posted image

Ghirik

Дата 18.2.2008, 13:44 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 919
Регистрация: 26.3.2007

Репутация: 13
Всего: 20

ksnk, спасибо.

Ну, блин!... задолбали браузеры. smile

--------------------

Отдадим всё клиенту, пускай его машина мучается...

Ghirik

Дата 19.2.2008, 01:32 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 919
Регистрация: 26.3.2007

Репутация: 13
Всего: 20

Посмотрите, как ответственно люди относятся к этой кнопке:

Код


/* 
*  Copyright 2006-2007 Dynamic Site Solutions.
*  Free use of this script is permitted for non-commercial applications,
*  subject to the requirement that this comment block be kept and not be
*  altered.  The data and executable parts of the script may be changed
*  as needed.  Dynamic Site Solutions makes no warranty regarding fitness
*  of use or correct function of the script.  Terms for use of this script
*  in commercial applications may be negotiated; for this, or for other
*  questions, contact "[email protected]".
*
*  Script by: Dynamic Site Solutions -- http://www.dynamicsitesolutions.com/
*  Last Updated: 2007-06-17
*/

//IE5+/Win, Firefox, Netscape 6+, Opera 7+, Safari, Konqueror 3, IE5/Mac, iCab 3

var addBookmarkObj = {
  linkText:'Bookmark This Page',
  addTextLink:function(parId){
    var a=addBookmarkObj.makeLink(parId);
    if(!a) return;
    a.appendChild(document.createTextNode(addBookmarkObj.linkText));
  },
  addImageLink:function(parId,imgPath){
    if(!imgPath || isEmpty(imgPath)) return;
    var a=addBookmarkObj.makeLink(parId);
    if(!a) return;
    var img = document.createElement('img');
    img.title = img.alt = addBookmarkObj.linkText;
    img.src = imgPath;
    a.appendChild(img);
  },
  makeLink:function(parId) {
    if(!document.getElementById || !document.createTextNode) return null;
    parId=((typeof(parId)=='string')&&!isEmpty(parId))
      ?parId:'addBookmarkContainer';
    var cont=document.getElementById(parId);
    if(!cont) return null;
    var a=document.createElement('a');
    a.href=location.href;
    if(window.opera) {
      a.rel='sidebar'; // this makes it work in Opera 7+
    } else {
      // this doesn't work in Opera 7+ if the link has an onclick handler,
      // so we only add it if the browser isn't Opera.
      a.onclick=function() {
        addBookmarkObj.exec(this.href,this.title);
        return false;
      }
    }
    a.title=document.title;
    return cont.appendChild(a);
  },
  exec:function(url, title) {
    // user agent sniffing is bad in general, but this is one of the times 
    // when it's really necessary
    var ua=navigator.userAgent.toLowerCase();
    var isKonq=(ua.indexOf('konqueror')!=-1);
    var isSafari=(ua.indexOf('webkit')!=-1);
    var isMac=(ua.indexOf('mac')!=-1);
    var buttonStr=isMac?'Command/Cmd':'CTRL';

    if(window.external && (!document.createTextNode ||
      (typeof(window.external.AddFavorite)=='unknown'))) {
        // IE4/Win generates an error when you
        // execute "typeof(window.external.AddFavorite)"
        // In IE7 the page must be from a web server, not directly from a local 
        // file system, otherwise, you will get a permission denied error.
        window.external.AddFavorite(url, title); // IE/Win
    } else if(isKonq) {
      alert('You need to press CTRL + B to bookmark our site.');
    } else if(window.opera) {
      void(0); // do nothing here (Opera 7+)
    } else if(window.home || isSafari) { // Firefox, Netscape, Safari, iCab
      alert('You need to press '+buttonStr+' + D to bookmark our site.');
    } else if(!window.print || isMac) { // IE5/Mac and Safari 1.0
      alert('You need to press Command/Cmd + D to bookmark our site.');    
    } else {
      alert('In order to bookmark this site you need to do so manually '+
        'through your browser.');
    }
  }
}

function isEmpty(s){return ((s=='')||/^\s*$/.test(s));}

function dss_addEvent(el,etype,fn) {
  if(el.addEventListener && (!window.opera || opera.version) &&
  (etype!='load')) {
    el.addEventListener(etype,fn,false);
  } else if(el.attachEvent) {
    el.attachEvent('on'+etype,fn);
  } else {
    if(typeof(fn) != "function") return;
    if(typeof(window.earlyNS4)=='undefined') {
      // to prevent this function from crashing Netscape versions before 4.02
      window.earlyNS4=((navigator.appName.toLowerCase()=='netscape')&&
      (parseFloat(navigator.appVersion)<4.02)&&document.layers);
    }
    if((typeof(el['on'+etype])=="function")&&!window.earlyNS4) {
      var tempFunc = el['on'+etype];
      el['on'+etype]=function(e){
        var a=tempFunc(e),b=fn(e);
        a=(typeof(a)=='undefined')?true:a;
        b=(typeof(b)=='undefined')?true:b;
        return (a&&b);
      }
    } else {
      el['on'+etype]=fn;
    }
  }
}

dss_addEvent(window,'load',addBookmarkObj.addTextLink);

// to make multiple links, do something like this:
/*
dss_addEvent(window,'load',function(){
  var f=addBookmarkObj.addTextLink;
  f();
  f('otherContainerID');
});
*/

// below is an example of how to make an image link with this
// the first parameter is the ID. If you pass an empty string it defaults to
// 'addBookmarkContainer'.
/*
dss_addEvent(window,'load',function(){
  addBookmarkObj.addImageLink('','/images/add-bookmark.jpg');
});
*/

Вот так подключается:

Код


<script type="text/javascript" src="/js/add_bookmark.js">
</script>
<p id="addBookmarkContainer"></p>

Это сообщение отредактировал(а) Ghirik - 19.2.2008, 01:41

--------------------

Отдадим всё клиенту, пускай его машина мучается...

artsb

Дата 3.11.2008, 09:59 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 2280
Регистрация: 17.7.2007
Где: центр Вселенной

Репутация: нет
Всего: 64

Хочу сделать универсальную ссылку для добавления в закладки. Делаю почти как автор топика.

Код


function bookmark()
{
    if (window.sidebar) {
        window.sidebar.addPanel(document.title, window.location,"");
    }  else if(window.opera&&window.print) {
          var mbm=document.createElement('a');
          mbm.setAttribute('href', window.location);
          mbm.setAttribute('title', document.title);
          mbm.setAttribute('onclick', 'window.external.AddFavorite(window.location, document.title); return false;');
          mbm.setAttribute('rel','sidebar');
         
          mbm.click();
    }  else if (document.all) {
        window.external.AddFavorite(window.location, document.title);
    }
}

В Opera не работает.
А если добавить такую ссылку:

Код


<a href="http://www.mysite.ru" title="Название сайта" onClick="window.external.AddFavorite(this.href, this.title); return false;" rel="sidebar">Добавить в избранное</a>

то всё ОК.
Но я не хочу для Opera'ы делать отдельную ссылку.
Как это сделать через JavaScript?

--------------------

Чем отличается умный человек от мудрого?
Умный - выпутается из любой ситуации.
Мудрый - просто в неё не попадёт.

Wolf1994

Дата 3.11.2008, 12:15 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1701
Регистрация: 5.10.2004

Репутация: 4
Всего: 29

Цитата(artsb @ 3.11.2008, 09:59

)

Но я не хочу для Opera'ы делать отдельную ссылку.
Как это сделать через JavaScript?

Определите тип браузера и в зависимости от него покажите ссылку. Определить браузер можно на сервере или на клиенте. На сервере, например на PHP, кажется, он определяется точнее, чем на JS.

bars80080

Дата 3.11.2008, 12:21 (ссылка)

(нет голосов)

Загрузка ...

прапор творюет

Награды: 1

Профиль
Группа: Завсегдатай
Сообщений: 12022
Регистрация: 5.12.2007
Где: Königsberg

Репутация: 15
Всего: 315

может переместить условие?

Код


   }  else if(window.print) {
        ...
    }  else if (window.opera||document.all) {
        window.external.AddFavorite(window.location, document.title);

Добавлено через 1 минуту и 21 секунду

Цитата(Wolf1994 @ 3.11.2008, 11:15

)

На сервере, например на PHP, кажется, он определяется точнее, чем на JS.

либо вообще не определяется.
имхо, лучше на js

artsb

Дата 3.11.2008, 15:32 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 2280
Регистрация: 17.7.2007
Где: центр Вселенной

Репутация: нет
Всего: 64

Wolf1994, я уже сделал как раз, как вы советуете. smile

bars80080, мне кажется дело не в условии, т.к. я с теми же условиями сейчас вывожу ссылки и всё работает.

--------------------

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| JavaScript: Общие вопросы \| Следующая тема »