Skip to content

Commit

Permalink
Rollup merge of rust-lang#46847 - GuillaumeGomez:more-sidebar-escape,…
Browse files Browse the repository at this point in the history
… r=QuietMisdreavus

Escape more items in the sidebar when needed

Fixes rust-lang#46724.

r? @QuietMisdreavus
  • Loading branch information
kennytm authored Dec 21, 2017
2 parents 614e285 + d4424d8 commit 256bf2b
Show file tree
Hide file tree
Showing 2 changed files with 49 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/librustdoc/html/render.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3718,9 +3718,10 @@ fn sidebar_assoc_items(it: &clean::Item) -> String {
})).and_then(|did| c.impls.get(&did));
if let Some(impls) = inner_impl {
out.push_str("<a class=\"sidebar-title\" href=\"#deref-methods\">");
out.push_str(&format!("Methods from {:#}&lt;Target={:#}&gt;",
impl_.inner_impl().trait_.as_ref().unwrap(),
target));
out.push_str(&format!("Methods from {}&lt;Target={}&gt;",
Escape(&format!("{:#}",
impl_.inner_impl().trait_.as_ref().unwrap())),
Escape(&format!("{:#}", target))));
out.push_str("</a>");
let ret = impls.iter()
.filter(|i| i.inner_impl().trait_.is_none())
Expand Down
45 changes: 45 additions & 0 deletions src/test/rustdoc/escape-deref-methods.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
// Copyright 2017 The Rust Project Developers. See the COPYRIGHT
// file at the top-level directory of this distribution and at
// http://rust-lang.org/COPYRIGHT.
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

#![crate_name = "foo"]

use std::ops::{Deref, DerefMut};

#[derive(Debug, Clone)]
pub struct Title {
name: String,
}

#[derive(Debug, Clone)]
pub struct TitleList {
pub members: Vec<Title>,
}

impl TitleList {
pub fn new() -> Self {
TitleList { members: Vec::new() }
}
}

impl Deref for TitleList {
type Target = Vec<Title>;

fn deref(&self) -> &Self::Target {
&self.members
}
}

// @has foo/struct.TitleList.html
// @has - '//*[@class="sidebar-title"]' 'Methods from Deref<Target=Vec<Title>>'
impl DerefMut for TitleList {
fn deref_mut(&mut self) -> &mut Self::Target {
&mut self.members
}
}

0 comments on commit 256bf2b

Please sign in to comment.