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

feat: Use limitInputPixels false #466

Merged
merged 1 commit into from
Feb 21, 2024
Merged

feat: Use limitInputPixels false #466

merged 1 commit into from
Feb 21, 2024

Conversation

kdheepak
Copy link
Contributor

This change is required because some gifs were too big to be preprocessed and causes errors during build:

withastro/astro#9546

This change is required because some gifs were too big to be preprocessed and causes errors during build:

withastro/astro#9546
@kdheepak kdheepak requested a review from joshka February 21, 2024 01:01
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1e43cb0
Status: ✅  Deploy successful!
Preview URL: https://20b27dc0.ratatui.pages.dev
Branch Preview URL: https://kdheepak-patch-1.ratatui.pages.dev

View logs

@kdheepak kdheepak merged commit b214193 into main Feb 21, 2024
8 checks passed
@kdheepak kdheepak deleted the kdheepak-patch-1 branch February 21, 2024 01:22
@joshka joshka mentioned this pull request Jun 4, 2024
joshka added a commit that referenced this pull request Jun 4, 2024
Although the `Stylize` trait is already implemented for `&str` which
extends to `String`, it is not implemented for `String` itself. This
commit adds an impl of Stylize that returns a Span<'static> for `String`
so that code can call Stylize methods on temporary `String`s.

E.g. the following now compiles instead of failing with a compile error
about referencing a temporary value:

    let s = format!("hello {name}!", "world").red();

BREAKING CHANGE: This may break some code that expects to call Stylize
methods on `String` values and then use the String value later. This
will now fail to compile because the String is consumed by set_style
instead of a slice being created and consumed.

This can be fixed by cloning the `String`. E.g.:

    let s = String::from("hello world");
    let line = Line::from(vec![s.red(), s.green()]); // fails to compile
    let line = Line::from(vec![s.clone().red(), s.green()]); // works
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant