Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix print styles #2974

Closed
kwalrath opened this issue Sep 7, 2019 · 5 comments · Fixed by #8042
Closed

Fix print styles #2974

kwalrath opened this issue Sep 7, 2019 · 5 comments · Fixed by #8042
Labels
e1-hours Effort: < 8 hrs infra.design Relates to the design of docs.flutter.dev infra.structure Relates to the tools that create docs.flutter.dev p3-low Valid but not urgent concern. Resolve when possible. Encourage upvote to surface.

Comments

@kwalrath
Copy link
Contributor

kwalrath commented Sep 7, 2019

Way too much is displayed when you print a flutter.dev page. I'm willing to fix this, unless someone else gets to it first. I did some similar work on dart.dev in https://github.com/dart-lang/site-www/pull/1625/files#diff-2f9b732d7e4f85168803a352c230300d.

We need to at least remove the left nav and the right-top icons. Also, I think lines are cut off. We also need to make sure that the TOC is handled properly. (It seems OK in the following screenshot, but it might not be OK for all pages.) The TOC should probably be displayed in full (rather than on the right or cut off), like in this screenshot.

image

@kwalrath kwalrath added infra.design Relates to the design of docs.flutter.dev e1-hours Effort: < 8 hrs p3-low Valid but not urgent concern. Resolve when possible. Encourage upvote to surface. labels Sep 7, 2019
@sfshaza2 sfshaza2 added the infra.structure Relates to the tools that create docs.flutter.dev label Sep 11, 2019
@micpap25
Copy link
Contributor

micpap25 commented Sep 7, 2020

Looks like this has been done. I think this can be closed.
image

@kwalrath
Copy link
Contributor Author

kwalrath commented Sep 8, 2020

Yes, you're right. I think I might've fixed this a long time ago!

@kwalrath kwalrath closed this as completed Sep 8, 2020
@kwalrath
Copy link
Contributor Author

kwalrath commented Sep 8, 2020

Oops, no, the problem is still there.

image

@kwalrath kwalrath reopened this Sep 8, 2020
@micpap25
Copy link
Contributor

micpap25 commented Sep 8, 2020

hmm, that's strange.
I'm using Windows 10, Google Chrome, newest version:
Version 85.0.4183.102 (Official Build) (64-bit)
Hopefully, that helps.

@kwalrath
Copy link
Contributor Author

kwalrath commented Sep 8, 2020

Strange! I tried the same page as you showed (https://flutter.dev/docs/development/accessibility-and-localization/internationalization) at various window widths (just in case), and I still see the left sidenav.

I'm using MacOS and tried Chrome Canary and 2 versions of Chrome stable:
Version 87.0.4258.2 (Official Build) canary (x86_64)
Version 85.0.4183.83 (Official Build) (64-bit)
Version 85.0.4183.102 (Official Build) (64-bit)

So... sounds like a Windows thing? That's really weird.

image

parlough added a commit to parlough/website that referenced this issue Jan 4, 2023
sfshaza2 pushed a commit that referenced this issue Jan 5, 2023
Fixes #2974

Co-authored-by: Brett Morgan <brettmorgan@google.com>
parlough added a commit to parlough/website that referenced this issue Jan 10, 2023
* Avoid extra border on embedded DartPads (flutter#8044)

Do so by moving the code-excerpt border to the pre elements used by snippets rather than the code-excerpt div itself.

* Bump site-shared from `444b310` to `faddba8` (flutter#8033)

Bumps [site-shared](https://github.com/dart-lang/site-shared) from `444b310` to `faddba8`.
- [Release notes](https://github.com/dart-lang/site-shared/releases)
- [Commits](dart-lang/site-shared@444b310...faddba8)

---
updated-dependencies:
- dependency-name: site-shared
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Parker Lougheed <parlough@gmail.com>

* Add overrides to fix print styles (flutter#8042)

Fixes flutter#2974

Co-authored-by: Brett Morgan <brettmorgan@google.com>

* Add redirect for old rendering doc (flutter#8047)

* Update mentions of Observatory to DevTools (flutter#8046)

* Update mentions of Observatory to DevTools

Also updates the instructions for opening DevTools

* Update connecting header

* Update name to Timeline events chart

* Add big juicy link to new codelab (flutter#8049)

* Add big link to the new codelab

* Remove files no longer needed

* Add an Oxford comma, naturally

* Clarify the info box at top of web codelab

* Fly-by fix of broken fragment links

* Bump examples/codelabs from `288ea9e` to `fc9dbfd` (flutter#8048)

* Scroll active sidenav entry into view on load (flutter#8050)

* updated banner for 17 days of Flutter (flutter#8052)

* Remove links to old “First app” codelab (flutter#8051)

* fix: typo (flutter#8053)

`a error` -> `an error` [link](https://owl.purdue.edu/owl/general_writing/grammar/articles_a_versus_an.html#:~:text=%22A%22%20goes%20before%20words%20that%20begin%20with%20consonants.&text=%22An%22%20goes%20before%20words%20that,an%20apricot)

Co-authored-by: Brett Morgan <brettmorgan@google.com>

* Fix applicationId mention and formatting in Android flavors section (flutter#8056)

* Fix a bunch of broken or outdated external links (flutter#8055)

* Fix non-response images on inspector page (flutter#8058)

Fixes flutter#6496

* Bump pigeon in /examples/development/platform_integration (flutter#8063)

Bumps [pigeon](https://github.com/flutter/packages/tree/main/packages) from 4.2.14 to 5.0.0.
- [Release notes](https://github.com/flutter/packages/releases)
- [Commits](https://github.com/flutter/packages/commits/pigeon-v5.0.0/packages)

---
updated-dependencies:
- dependency-name: pigeon
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update firebase.json (flutter#8067)

Update link destination for pull-request-revert-api-v2. Previous link was not working correctly.

* Fix sidebar collapse icon alignment (flutter#8068)

* Enable Jekyll frontmatter and some liquid validation (flutter#8059)

Fixes flutter#3489

* Bump actions/setup-node from 3.5.1 to 3.6.0 (flutter#8061)

Bumps [actions/setup-node](https://github.com/actions/setup-node) from 3.5.1 to 3.6.0.
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](actions/setup-node@8c91899...64ed1c7)

---
updated-dependencies:
- dependency-name: actions/setup-node
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump actions/checkout from 3.2.0 to 3.3.0 (flutter#8060)

Bumps [actions/checkout](https://github.com/actions/checkout) from 3.2.0 to 3.3.0.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@755da8c...ac59398)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Split Flutter for iOS developers into pages for SwiftUI and UIKit (flutter#8070)

* Split Flutter for iOS developers into page for SwiftUI and UIKit

* Fix redirect link

* Update some links to new articles

* Remove extra SwiftUI link

* Adding Parker (flutter#8074)

* Fixed mobile layout on 'Creating responsive and adaptive apps' page (flutter#8073)

Updated the the mobile responsiveness of a video url.

Co-authored-by: Parker Lougheed <parlough@gmail.com>

* Fix incorrect migration sample in RenderObjectElement migrations (flutter#8078)

* Fix formatting of 'Unsupported platforms' table (flutter#8079)

* Fix first web app codelab returning private state lint (flutter#8081)

* Fix multiple API doc links (flutter#8082)

* Fix images not staying in content on 'Building adaptive apps' (flutter#8077)

Fixes flutter#7470

* Uncomment post-publish SwiftUI doc link (flutter#8075)

* feat/solidart-state-management-option (flutter#8071)

Add `solidart` as a state management option

* Add go link for Platinum Tests (flutter#8069)

Co-authored-by: Brett Morgan <brettmorgan@google.com>

* Bump actions/upload-artifact from 3.1.1 to 3.1.2 (flutter#8085)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3.1.1 to 3.1.2.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@83fd05a...0b7f8ab)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Followup adjustments to first Flutter app codelab (flutter#8087)

* Fix “Building layouts (step 0)”

* Fix “Adding interactivity (step 0)”

* Update examples/layout/base/lib/main.dart

Co-authored-by: Parker Lougheed <parlough@gmail.com>

* Regenerate

Co-authored-by: Parker Lougheed <parlough@gmail.com>

* Finalize followup adjustments for the new "First Flutter app" codelab (flutter#8088)

This updates the following:

- [Testing accessibility on mobile](https://docs.flutter.dev/development/accessibility-and-localization/accessibility#testing-accessibility-on-mobile) relying on and referencing the old, created app and the [Accessibility Guideline API](https://api.flutter.dev/flutter/flutter_test/AccessibilityGuideline-class.html)

_Issues fixed by this PR (if any):_ Fixes flutter#8054

Co-authored-by: Parker Lougheed <parlough@gmail.com>

* Bump diff2html-cli from 5.2.7 to 5.2.9 (flutter#8084)

Bumps [diff2html-cli](https://github.com/rtfpessoa/diff2html-cli) from 5.2.7 to 5.2.9.
- [Release notes](https://github.com/rtfpessoa/diff2html-cli/releases)
- [Commits](rtfpessoa/diff2html-cli@5.2.7...5.2.9)

---
updated-dependencies:
- dependency-name: diff2html-cli
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Clean up and fix some issues on 'Android views' page (flutter#8089)

* Cleans up and fixes some issues on Android platform views page

* Fix leading whitespace

* Update internationalization documentation (flutter#8065)

* init

* more changes + code excerpts

* revert package lock? didn't change any packages so idk why this is updating

* add l10n config info

* fix issues

* update code snippets?

* fix

* fix code snippets

* fix stable

* finally fixed

* fix everything

* Update src/development/accessibility-and-localization/internationalization.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update src/development/accessibility-and-localization/internationalization.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update src/development/accessibility-and-localization/internationalization.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update src/development/accessibility-and-localization/internationalization.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update src/development/accessibility-and-localization/internationalization.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update src/development/accessibility-and-localization/internationalization.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update examples/internationalization/gen_l10n_example/lib/examples.txt

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update examples/internationalization/gen_l10n_example/lib/main.dart

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update src/development/accessibility-and-localization/internationalization.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update src/development/accessibility-and-localization/internationalization.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* fix some stuff

* Apply suggestions from code review

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* add comment about case-sensitive-ness of select comparisons

* Apply suggestions from code review

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* fix code-excerpts

* fix code-excerpt issues

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Replace outdated links to site-shared writing wiki (flutter#8092)

* Make the example visually explicit (flutter#8083)

Make the example visually explicit with the native android reference: https://developer.android.com/training/multiscreen/screendensities#TaskProvideAltBmp

* Update pull request template for website freeze (flutter#8041)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Parker Lougheed <parlough@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Brett Morgan <brettmorgan@google.com>
Co-authored-by: Filip Hracek <filiph@users.noreply.github.com>
Co-authored-by: Shamira <23406866+MiraMarshall@users.noreply.github.com>
Co-authored-by: Claire Davis <clairepdavis@gmail.com>
Co-authored-by: Ricardo Amador <32242716+ricardoamador@users.noreply.github.com>
Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>
Co-authored-by: AbdulHafeezGalib <77202232+Abdul-Hafeez-Galib@users.noreply.github.com>
Co-authored-by: Alexandru Mariuti <alexandrumariuti@protonmail.com>
Co-authored-by: pdblasi-google <109253501+pdblasi-google@users.noreply.github.com>
Co-authored-by: Tae Hyung Kim <thkim1011@users.noreply.github.com>
Co-authored-by: David Gaspar <davidgaspar.dev@gmail.com>
Co-authored-by: John Ryan <ryjohn@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e1-hours Effort: < 8 hrs infra.design Relates to the design of docs.flutter.dev infra.structure Relates to the tools that create docs.flutter.dev p3-low Valid but not urgent concern. Resolve when possible. Encourage upvote to surface.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants