Hi @smustgrave,
I have checked this issue on versions 9.4.0-dev, 10.0.12-dev, 11.x-dev, but it is not reproducible on my end. I have attached screenshots for your reference.
I have followed all the steps mentioned here.
- Setup Drupal
- Activate Claro theme and set default theme
- Create node with long title
- Go to incognito tab with created node view page
- There line-height between breadcrumbs and title is not properly showing And Node title and header padding not properly showing.
Please let me know what should be the next step from my side.
Thank you!
Hi,
I tried to check the issue with steps mentioned in issue description on Drupal 11, but unable to replicate the issue.
Below I have attached screenshots on desktop and mobile view. As, if the issue is not reproducible on latest version can we close this case?
I am moving this to Needs Review.
Thanks!
Hi,
Changed title from 'Do not require a 'title' field' to 'Remove 'not null' constraint from node title field/column.('Title' field present in node entity should not be mandatory.)'.
Thanks!
Hi,
While going through issue, got reference of linked issue ( https://www.drupal.org/project/drupal/issues/3347648 ๐ Remove @todo-comment linking to closed issue in layout-builder.css Closed: duplicate ).
There we have suggession to remove only @todo-comment linking. So removed only comment line and raised merge request(https://git.drupalcode.org/project/drupal/-/merge_requests/9763) for that.
@smustgrave As there is no UI change or any such change do we need to update summary? Please let me know.
Thanks!
supriyak โ made their first commit to this issueโs fork.
supriyak โ made their first commit to this issueโs fork.
Hi,
MR has been created !10005(https://git.drupalcode.org/project/drupal/-/merge_requests/10005).
Written testcase as per requirement. Please let me know incase of any new changes needs to be add.
Thanks!
supriyak โ made their first commit to this issueโs fork.
Hi @anybody,
Below are my findings and observations while working on this issue.
1. After enabling 'List.js' and 'Listjs views' modules when I try to edit any view and changed 'Format' to 'Listjs', it is giving me below error.
=> Error: Call to a member function __toString() on string in Drupal\listjs_views\Plugin\views\style\Listjs->buildOptionsForm() (line 74 of /var/www/html/modules/contrib/listjs/modules/listjs_views/src/Plugin/views/style/Listjs.php).
This error got resolved after removing '->__toString();' from line number 74.
Will you please let me know that can we create another issue regarding this error?
2. I have used 'Listjs' format in 'admin/content' view where it seems to be fuzzy search is working already. Below I have attached screen recording for it.
Will you please let me know that do we need to keep this issue open or we can close this? Or am I missing/misunderstood this issue? Please let me know so that I will get clarity on this and can work on it.
Thank you!!
supriyak โ made their first commit to this issueโs fork.
Hi,
After patch application still there are so many errors/warnings. So, moving this to 'Needs Work'.
Thanks!
Hi @sarwan_verma,
After patch apply I am getting below error, here I have attached screenshot FYI. Moving this to needs work.
Thanks!
Hi,
Please find below before and after patch screenshots. Before applying patch conditions were not working and after applying patch those are working. RTBC from my end.
Thanks!
Hi,
I have executed generate-theme command using starterkit. Below are the screenshots of template files before and after patch. We can see here 'description display before/after conditionals' in after-patch screenshot.
Thanks!
Hi,
I have executed generate-theme command using starterkit. Below are the screenshots of template files before and after patch. We can see here 'description display before/after conditionals' in after-patch screenshot.
Thanks!
Hi @Anybody,
Please review my changes and let me know your valuable feedback on this.
MR - https://git.drupalcode.org/project/watchdog_mailer/-/merge_requests/23
Commit - https://git.drupalcode.org/project/watchdog_mailer/-/merge_requests/23/d...
Thank you!
Thank you @Anybody for reply.
Can you please check 'WatchdogMailerSettingsTest.php'? In this file there are no levels and sublevels, here they maintain sequence only.
So it will be confusing here. Please suggest on this.
I have done with my code just waiting for your reply on this.
Thanks!!
Hi @Anybody,
"enabled" key is already present in watchdog_mailer.settings for another field. It is affecting to changing key for 'watchdog_mailer_enable'.
Please suggest on this.
Thanks!
supriyak โ made their first commit to this issueโs fork.
supriyak โ made their first commit to this issueโs fork.
Hi,
I have tested this from my end. Moving this issue in RTBC.
Thanks!
Hi smustgrave,
Please find updated MR : https://git.drupalcode.org/project/drupal/-/merge_requests/8470
Thank You!!
Hi @smustgrave,
MR is updated.
MR Link : https://git.drupalcode.org/project/drupal/-/merge_requests/8470
Thank you!!
supriyak โ made their first commit to this issueโs fork.
Hi Team,
Please review MR - https://git.drupalcode.org/project/list_predefined_options/-/merge_reque....
Thanks!!
Hi @DamienMcKenna,
I think this theme is not compiled yet.
I have tried to compile the files here. The data in table.css and table.scss is different.
So when I try to compile table.scss, previous data of that file got removed and only new changes are there which is not acceptable.
Please suggest me should I add the code in both the files?
Thanks!!
Hi @DamienMcKenna,
Thank you for your reply. I have pushed my new changes in same MR.
Please let me know your feedback on this.
Hi Everyone,
This bug is occurring in 'Bootstrap Barrio Subtheme'.
To fix this bug I have raised MR : https://git.drupalcode.org/project/bootstrap_barrio/-/merge_requests/83. Please review it.
Thank you!!
supriyak โ made their first commit to this issueโs fork.
Hi @21kPiyush,
I have tried to reproduce the alert message on my local setup, but I am not getting it.
As you said when I am editing the webform submission the selected Date and time slot is coming in webform_booking element but it is not selected on calendar.
So we can create the task to do this functionality. Please let me know still you getting same alert message? And if yes let me know steps to reproduce it.
Thanks!
Hi,
Applied patch successfully which is commented on '#2'. Issue is resolved using this patch. Attaching after and before screenshots.
@Muskan0215 Instead of using external CSS have you checked any other solution here?
Thanks!
Tested patch '#2' and 'MR !8' in #6.
Checking patch docroot/modules/contrib/email_confirmer/email_confirmer.info.yml...
Checking patch docroot/modules/contrib/email_confirmer/email_confirmer_user/email_confirmer_user.info.yml...
Applied patch docroot/modules/contrib/email_confirmer/email_confirmer.info.yml cleanly.
Applied patch docroot/modules/contrib/email_confirmer/email_confirmer_user/email_confirmer_user.info.yml cleanly.
Both the patches are successfully applying and changes are reflected.
Please find below before and after screenshots for same.
RTBC from me.
I am migrating some sites from D9 to D10 for that I need to make content_sync compatible with D10.
I tried to apply #9, #20 and #22 patches in my code, but can not make it success. So, creating another one.
Here is a patch based on status_upgrade checks and previous patches here.
It is for 3.0.0-beta1 version.