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

4.0 Compatibility #57

Closed
afbora opened this issue Jun 22, 2023 · 8 comments
Closed

4.0 Compatibility #57

afbora opened this issue Jun 22, 2023 · 8 comments

Comments

@afbora
Copy link
Contributor

afbora commented Jun 22, 2023

Tested on 4.0.0-alpha.3 and working great! There is just minor UI issue:

  • Dialog UI

screen-capture-1610-Test - Mægazine-localhost

  • Reset button position
  • Empty text icon size
@nmanngit
Copy link

I just created #58 to address the dialog UI and resolve a breaking change introduced in alpha 4. While working on it, I also encountered some more UI bugs. However, I'm unsure whether these should be handled within this repo or reported to the Kirby repo. They happen due to these CSS overrides:

/* Causes locator icon to shrink which feels unintuitive */
:where(img,picture,svg) {
    max-inline-size: 100%;
}

/* Causes leaflet flag icon to move to a separate line. Note leaflet is an external dependency */
:where(img,svg,video,canvas,audio,iframe,embed,object) {
    display: block;
}

What do you think?

@bnomei
Copy link

bnomei commented Sep 30, 2023

ping @sylvainjule

@bnomei
Copy link

bnomei commented Oct 2, 2023

constructor of structure changed. your fieldMethods.php needs a fix

<?php

use Kirby\Cms\Structure;

return [
    'toLocation' => function ($field) {
        $structure = new Structure([$field->yaml()], [
            'parent' => $field->parent()
        ]);

        return $structure->first();
    },
];

@nmanngit
Copy link

nmanngit commented Oct 2, 2023

@bnomei please have a look at my PR, where I used Structure::factory() as mentioned here.

@ovenum
Copy link

ovenum commented Oct 9, 2023

Just noticed that the Field Preview requires some tweaks and maybe rounded borders to be inline with the current Panel styles.

@sylvainjule do you prefer issues being created here or as separate github issues? Also happy to create pull requests if needed.

image

@sylvainjule
Copy link
Owner

Thank you for this and the PR! I'm catching up with maintenance of all my plugins this week, once they're all up to speed for Kirby 3.9 I'll dig into 4.0.

@renestalder
Copy link
Contributor

Apart from visual quirks, does it otherwise work with v4? Does anybody use it without the fixes?

@sylvainjule
Copy link
Owner

The plugin is now compatible with Kirby 4 ✓
I'm closing this, I've tried to review all options / use cases, don't hesitate to open dedicated issues if you have trouble with this latest release.

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

No branches or pull requests

6 participants