diff options
| author |  Alexis Degrugillier <github@ainw.org>
| 2017-06-02 22:53:13 +0200 |
|---|---|---|
| committer | Alexis Degrugillier <github@ainw.org>
| 2017-06-02 22:53:13 +0200 |
| commit | 5f942cc1c9801eef440cd1e85e36b03c05aaa8c5 (patch) | |
| tree | ee019ee6f7738f84cdc83b64c074d82d93e9f8ce | |
| parent | d100b2deb1c8ddc7fb8422abe420e2d135c6a8e2 (diff) | |
Add support for a new shortcut in reader view
Now you can open the original page in the reader view with the same shortcut you'll use in the normal view.
I've changed how we identify the link to make it more flexible.
The previous way was too restrictive since the selector used a really strict path to get the url.
There was another way to achieve the same thing without changing the selector.
It was quite ugly since some meaningless class would be added on the markup to match the selector query.
See #1400
| -rw-r--r-- | app/views/index/normal.phtml | 2 | ||||
| -rw-r--r-- | app/views/index/reader.phtml | 2 | ||||
| -rw-r--r-- | p/scripts/main.js | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/app/views/index/normal.phtml b/app/views/index/normal.phtml index 6fda11ed9..ba48b2501 100644 --- a/app/views/index/normal.phtml +++ b/app/views/index/normal.phtml @@ -66,7 +66,7 @@ if (!empty($this->entries)) { ?><div class="flux_content"> <div class="content <?php echo $content_width; ?>"> - <h1 class="title"><a target="_blank" rel="noreferrer" href="<?php echo $this->entry->link(); ?>"><?php echo $this->entry->title(); ?></a></h1> + <h1 class="title"><a target="_blank" rel="noreferrer" class="go_website" href="<?php echo $this->entry->link(); ?>"><?php echo $this->entry->title(); ?></a></h1> <?php $author = $this->entry->author(); echo $author != '' ? '<div class="author">' . _t('gen.short.by_author', $author) . '</div>' : '', diff --git a/app/views/index/reader.phtml b/app/views/index/reader.phtml index a19ee322e..f2af75af0 100644 --- a/app/views/index/reader.phtml +++ b/app/views/index/reader.phtml @@ -19,7 +19,7 @@ if (!empty($this->entries)) { $feed = FreshRSS_CategoryDAO::findFeed($this->categories, $item->feed()); //We most likely already have the feed object in cache if (empty($feed)) $feed = $item->feed(true); ?> - <a href="<?php echo $item->link(); ?>"> + <a target="_blank" rel="noreferrer" class="go_website" href="<?php echo $item->link(); ?>"> <img class="favicon" src="<?php echo $feed->favicon(); ?>" alt="✇" /> <span><?php echo $feed->name(); ?></span> </a> <h1 class="title"><?php echo $item->title(); ?></h1> diff --git a/p/scripts/main.js b/p/scripts/main.js index f01af20ff..78a3772f6 100644 --- a/p/scripts/main.js +++ b/p/scripts/main.js @@ -661,7 +661,7 @@ function init_shortcuts() { }); shortcut.add(shortcuts.go_website, function () { - var url_website = $('.flux.current > .flux_header > .title > a').attr("href"); + var url_website = $('.flux.current a.go_website').attr("href"); if (context.auto_mark_site) { $(".flux.current").each(function () { |