fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] changelog2011-04-19 10:16 am

[dw-free] /icons should have "site" as option in nav strip

[commit: http://hg.dwscoalition.org/dw-free/rev/cc44ea023fcc]

http://bugs.dwscoalition.org/show_bug.cgi?id=3606

Show style=site link in nav strip for icons page.

Patch by [personal profile] fu.

Files modified:
  • bin/upgrading/s2layers/core2.s2
  • cgi-bin/weblib.pl
--------------------------------------------------------------------------------
diff -r 5cc8c72df9c4 -r cc44ea023fcc bin/upgrading/s2layers/core2.s2
--- a/bin/upgrading/s2layers/core2.s2	Tue Apr 19 17:52:11 2011 +0800
+++ b/bin/upgrading/s2layers/core2.s2	Tue Apr 19 18:15:56 2011 +0800
@@ -1202,7 +1202,7 @@ set use_journalstyle_entry_page = true;
 set use_journalstyle_entry_page = true;
 
 property bool use_journalstyle_icons_page  { des = "Show icons page in my journal style rather than the site layout"; }
-set use_journalstyle_icons_page = false;
+set use_journalstyle_icons_page = true;
 
 # Defaults to false, but layouts can set it to true to switch on custom colors for different posters on reading list.
 property bool use_custom_friend_colors  { des = "Use my custom reading list colors"; }
diff -r 5cc8c72df9c4 -r cc44ea023fcc cgi-bin/weblib.pl
--- a/cgi-bin/weblib.pl	Tue Apr 19 17:52:11 2011 +0800
+++ b/cgi-bin/weblib.pl	Tue Apr 19 18:15:56 2011 +0800
@@ -3115,7 +3115,11 @@ LOGIN_BAR
         if ( $view_type eq "mine" and $current_style ne $view_type and $remote and not $remote->equals( $journal ) ) {
             push @view_options, "<a href='" . $make_style_link->( $view_type ) . "'>" .
                 LJ::Lang::ml( 'web.controlstrip.reloadpage.mystyle2' ) . "</a>";
-        } elsif ( $view_type eq "site" and $current_style ne $view_type and ( $view eq "entry" or $view eq "reply" ) ) {
+        } elsif ( $view_type eq "site" and $current_style ne $view_type and {
+                entry => 1,
+                reply => 1,
+                icons => 1,
+            }->{$view} ) {
             push @view_options, "<a href='" . $make_style_link->( $view_type ) . "'>" .
                 LJ::Lang::ml( 'web.controlstrip.reloadpage.sitestyle' ) . "</a>";
         } elsif ( $view_type eq "light" and $current_style ne $view_type ) {
--------------------------------------------------------------------------------

Post a comment in response:

This account has disabled anonymous posting.
If you don't have an account you can create one now.
HTML doesn't work in the subject.
More info about formatting

If you are unable to use this captcha for any reason, please contact us by email at support@dreamwidth.org