UI: Styling for Published Page (PR #10335)

Styling to look more visually connected to Discourse

Before

image

After

image

GitHub

You might want to reuse the logic we have in application_helper:

  def application_logo_url
    @application_logo_url ||= begin
      if mobile_view? && SiteSetting.site_mobile_logo_url.present?
        SiteSetting.site_mobile_logo_url
      else
        SiteSetting.site_logo_url
      end
    end
  end

You might want to reuse the logic we have in application_helper:

  def application_logo_url
    @application_logo_url ||= begin
      if mobile_view? && SiteSetting.site_mobile_logo_url.present?
        SiteSetting.site_mobile_logo_url
      else
        SiteSetting.site_logo_url
      end
    end
  end

do we always have a name ? I’m unsure

should be $secondary no ?

should maybe use the scss helper for z-index ?

I did try this out, but it returns the full width site logo on desktop. I want the small/mobile logo to appear here regardless of desktop/mobile user.

image

@jjaffeux suggestions implemented

as I hate inline style, something we could at least do here is only having border-color :stuck_out_tongue: and move width/style in css

Looks very nice, thank you!