🇨🇦Canada @man-1982

chadhester → credited man-1982 → .

Reviewed and tested.
Drupal 10.3
MR was applied without any issue.
Suggest adding MR
thanks

man-1982 → made their first commit to this issue’s fork.

i can reproduce it only if before i had old configuration from previous version "Swiper formatter"
If you have fresh new version Swiper formatter and a create new teplate profile it can't be reproduce.

Close that ticket as won't be reproduce

thanks

man-1982 → created an issue.

@nk_, @bbu23 Thank you for your work

Close this ticked by know.

We need to focus on testing the new module's features.

Ready to review

It works
Suggest add this functionality to the module

man-1982 → created an issue.

Hi!
New features looks really cool and promises.
going to test lates dev version.
thanks

What's been added:

Drupal modal dialog or native HTML5 dialog option
Controller and route for generating the original Swiper

Hi gays,

i'de like to ask what we decided about patch?
Do we need more extra tests or works?

it would be my pleasure to help with this one.

thanks

Added a few more images:
1:

2:

3:

As proof of works please consider this gif-video

There is now an option to select another format for the swiper field.

The result of the work is the ability to open a Swiper by clicking on the content teaser, and the swiper will open in a Drupal modal window on the exact slider where the click occurred.

Added cashing base on a tags.

Thank you

Report:
1. I added bad Elasticsearch conf, which doesn work on my local env
2. I had this error

3. Applied patch through composer in composer patches part

"drupal/elasticsearch_connector": {
                "produce fatal error when backend is down": "https://git.drupalcode.org/project/elasticsearch_connector/-/merge_requests/63.patch"
            }

4. Patch was applied without problem, all passed smoothly
5. as a result a can see list of my servers and index

I strongly recommend apply this changes to module.
It seems to me, there is a critical issue, because we don't have access to servers search list without this patch

thanks to
@kallevu
@sokru

Hi @smustgrave!
Could you provide a couple examples or give me direction, where i can have a look and read about

type hints should be included

I didn't clear understand your message.

Thank you.

man-1982 → changed the visibility of the branch 3469036-add-support-for to active.

man-1982 → changed the visibility of the branch 3469036-add-support-for to hidden.

@erik.erskine
Sorry, I double-checked. You are right. There is no need extra router files.
I've made some changes in MR:

• Added configure route
• Added some simple functional tests

Thanks

we have daterange_compact.links.menu.yml:

entity.daterange_compact_format.collection:
  title: 'Compact date and time range formats'
  route_name: entity.daterange_compact_format.collection
  description: 'Configure how date and time ranges are displayed in a compact form.'
  parent: system.admin_config_regional
  weight: 0

it's mean that we expect in url /admin/config/regional/daterange-compact-format (=entity.daterange_compact_format.collection) entities list collection
instead we have "Page not found"
I attached a couple screenshot to be more clarified.

Hi everyone,

You do however get a very long scroll bar

@kevinvb you are absolutely right. But in my defenss, i did try a few other approaches (position:static, relative and one more), but this was the only one that actually only worked for me.
@kevinvb i'm alway open for new idea, so if you've got one please let me know. It would be perfect.

@andreastkdf (Andrea Kostakis) special thanks to you, you've put a lot of work. Great job Andrea!

Hi @kevinvb
I've noticed really odd situation. When i tried to apply patch from https://git.drupalcode.org/project/gin/-/merge_requests/459.patch it didn't
work.
However when i applied patch from https://git.drupalcode.org/project/gin/-/merge_requests/459.diff. it were applied without any issue.
I use composer for applying patches.
You can see result on the picture

@kevinvb Hi!

is it possible to provide a patch on the latest release?

It's really interesting and i'm not clear understand how it's work, but i used

"patches": {
            "drupal/gin": {
              "[3242554] Contents of dropdown button go to the left and can be off the visible page": "https://git.drupalcode.org/project/gin/-/merge_requests/459.patch"
            }
        } 

in my composer.json and i can see the patching result in my code. (of course i've deleted gin folder and composer.lock before composer install)
However i see some errors in console during composer install executing, but again i see patch was applied.

[Exception]                                                                                                                                                                   
  Cannot apply patch [3242554] Contents of dropdown button go to the left and can be off the visible page (https://git.drupalcode.org/project/gin/-/merge_requests/459.patch)!                                                                                                                                                          

Also i see Andrea Kostakis (@andreastkdf) added lots of changes in his last commit.

@andreastkdf Hi!
Could you take a loot to your lates commit. I see we can't pass some frontend

Added fixes in css styles and re-worked some JS behaviour.
Ready to review.
Tested on desktop and on mobile too
I've made small movies as a proof, that MR works and can be merge

i used @andreastkdf `s MR and added my changes.
@andreastkdf you've done great work. Thanks

man-1982 → made their first commit to this issue’s fork.

I've prepared some proof that MR does work. I used clean installation Drupal 11.

thanks

Looks like MR ready to review!
Dear Colleagues please you take a look.
Thanks

man-1982 → created an issue.

man-1982 → made their first commit to this issue’s fork.

man-1982 → made their first commit to this issue’s fork.

But the actual cookie was with SSESS

I can't force to work my tests
@MykolaVeryha thanks a lots.
You save me.

Looks like fixed
as a proof
you can see small video

@jv24
yep, correct. I used the same approach in my MR.
Looks like all works good
Thanks.

man-1982 → created an issue.

Created Merge Request.
Works good
as a proof attached gif-video

Please review.
Thanks

man-1982 → created an issue.

This patch aims to enhance the daterange_compact module by ensuring that its configuration pages are accessible and function correctly. It also adds a layer of automated testing for future development and maintenance.

Please review the attached patch and provide any feedback or additional testing from the community.

Added daterange_compact.routing.yml and Functional Test DateRangeCompactRoutingTest.php

man-1982 → created an issue.

Hello everyone,
I'm interested in becoming a maintainer for the Formatter Field → project.

Our company has used this module from Drupal 7, and I've contributed by providing patches for issues:
Issue #3406394 🐛 Formatter Field Not Functioning Correctly in Paragraphs on Drupal 10 Needs review
Issue # 3260320 🐛 TypeError: Argument 1 passed to Drupal\formatter_field\Plugin\Field\FieldFormatter\FromFieldFormatter::getFormatter() RTBC

I've been working with Drupal for over 13 years, starting from Drupal 6 up to the current Drupal 10.
This experience has given me a strong background in developing and customizing modules for various Drupal versions.

I have several ideas to improve the module:

• Adde support for third-party display settings.
• Create a configuration entity for saving settings based on entity type and fields.
• Develop a migration config/submodule from Drupal 7 to Drupal 10.
• Implement and follow security best practices.

I am also ready to take on the responsibilities that come with security coverage.
I believe my technical skills and my dedication to the Drupal community make me a good fit for maintaining this module..

Thank you for considering my application.

Best,
Andrii Kocherhin →

Hi everyone
I've created patch which can fix this issue.
you can see it here
https://www.drupal.org/project/formatter_field/issues/3406394 🐛 Formatter Field Not Functioning Correctly in Paragraphs on Drupal 10 Needs review
Thanks

I've added a patch to resolve this issue. The patch is also compatible with formatter_field-3260320-4.patch → from previous topic 🐛 TypeError: Argument 1 passed to Drupal\formatter_field\Plugin\Field\FieldFormatter\FromFieldFormatter::getFormatter() RTBC

You can view the results in this screencast: Screencast from 13-12-23 at 12:22:07 → .

man-1982 → created an issue.

I noticed an issue: when attempting to edit or add a field in the "Manage fields" section, I consistently encountered an error. The error message displayed was:

The website encountered an unexpected error. Please try again later.

TypeError: array_intersect_key(): Argument #1 ($array) must be of type array, bool given in array_intersect_key() (line 158 of core/lib/Drupal/Core/Field/FormatterPluginManager.php).

To resolve this issue, I applied a patch which successfully addressed the problem.

My patch extends the patch by @validoll.
Thanks

Test Environment:

Drupal version: 10.1.6
DB driver: MySQL
PHP OS: Linux
PHP version: 8.1.25

Results:

The patch successfully resolved the issue described. All works good.
Suggested @JurriaanRoelofs to add patch to main branch.
thanks

The patch in #17 functions as expected. It was tested with the following configurations:

Drupal version: 10.1.6
PHP version: 8.1.25
Additionally, the latest development version of the module was used: composer require 'drupal/focal_point:2.x-dev@dev'.

Suggest to add this patch into main branch.

Thank you.

All works good!
The "Entity embed" dev with this patch( https://www.drupal.org/project/entity_embed/issues/3272732 ✨ Drupal 10 & CKEditor 5 readiness Fixed ) can't work without this patch.
Please merge its.

more right patch

man-1982 → created an issue.