Skip to content

Conversation

jsha
Copy link
Contributor

@jsha jsha commented Dec 9, 2021

This fixes an issue when displaying on iPad, where the search box had no borders.

r? @GuillaumeGomez

Demo https://rustdoc.crud.net/jsha/webkit-appearance-search-input/std/string/struct.String.html

@jsha jsha added T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. A-rustdoc-ui Area: Rustdoc UI (generated HTML) labels Dec 9, 2021
@rust-highfive
Copy link
Contributor

Some changes occurred in HTML/CSS/JS.

cc @GuillaumeGomez

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 9, 2021
@GuillaumeGomez
Copy link
Member

An ipad-specific issue, it's been a while. :)

Can you add a comment on the CSS property please? After that, r=me

@camelid camelid changed the title Add -webkit-appearance: none to search input Add -webkit-appearance: none to search input Dec 9, 2021
@jsha jsha force-pushed the webkit-appearance-search-input branch from b2784d5 to a148c69 Compare December 10, 2021 07:07
@rust-log-analyzer

This comment has been minimized.

@jsha jsha force-pushed the webkit-appearance-search-input branch from a148c69 to 7c1b7ea Compare December 10, 2021 18:23
@rust-log-analyzer

This comment has been minimized.

This fixes an issue when displaying on iPad, where the search box had no
borders.
@jsha jsha force-pushed the webkit-appearance-search-input branch from 7c1b7ea to da472a5 Compare December 13, 2021 04:33
@GuillaumeGomez
Copy link
Member

Thanks!

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented Dec 13, 2021

📌 Commit da472a5 has been approved by GuillaumeGomez

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 13, 2021
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Dec 13, 2021
…, r=GuillaumeGomez

Add `-webkit-appearance: none` to search input

This fixes an issue when displaying on iPad, where the search box had no borders.

r? `@GuillaumeGomez`

Demo https://rustdoc.crud.net/jsha/webkit-appearance-search-input/std/string/struct.String.html
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 13, 2021
…askrgr

Rollup of 5 pull requests

Successful merges:

 - rust-lang#91699 (Add `-webkit-appearance: none` to search input)
 - rust-lang#91846 (rustdoc: Reduce number of arguments for `run_test` a bit)
 - rust-lang#91847 (Fix FIXME for `generic_arg_infer` in `create_substs_for_ast_path`)
 - rust-lang#91849 (GATs outlives lint: Try to prove bounds)
 - rust-lang#91855 (Stabilize const_cstr_unchecked)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit e354f0c into rust-lang:master Dec 13, 2021
@rustbot rustbot added this to the 1.59.0 milestone Dec 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rustdoc-ui Area: Rustdoc UI (generated HTML) S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants