Problem/Motivation
URL language detection is by default not available via the interface. Once enabled via configuration PHP Notices are shown and the language detection title and description are incomplete.
To reproduce:
- Enable Language module
- Export configuration
- Edit the file 'language.types.yml' and edit. Add 'language_url' to the 'configurable' section as below.
configurable:
- language_interface
- language_content
- language_url
- Re-import the configuration
- Go to /admin/config/regional/language/detection
- Notice the Message area and the section "language detection" near the bottom of the page.
Proposed resolution
Add title and description for the URL language detection.
Remaining tasks
t.b.d.
User interface changes
Completed title and description for URL language detection.
Comment | File | Size | Author |
---|---|---|---|
#17 | 2824086-17-language-url-detection.patch | 2.26 KB | analiam319@gmail.com |
#9 | language-url-detection-2824086-9.patch | 685 bytes | manuel.adan |
#2 | language-url-detection-2824086-2.patch | 690 bytes | Sutharsan |
#2 | language-url-detection-title-2824086.png | 171.18 KB | Sutharsan |
#2 | language-url-detection-message-2824086.png | 44.54 KB | Sutharsan |
Comments
Comment #2
Sutharsan CreditAttribution: Sutharsan commentedComment #3
Sutharsan CreditAttribution: Sutharsan commentedComment #5
markhuot CreditAttribution: markhuot as a volunteer commentedIs there any desire to get this merged in? I'm happy to write tests or adjust config if there's a desire.
From my point of view there this is vital to using language negotiators. I have a custom language negotiator that is working well for the Admin node/edit and node/translate views but breaks when trying to view node/123 because my custom negotiator isn't called.
Comment #9
manuel.adanPatch #2 did not apply in 8.5/8.6.x, ported (interdiff 2-9 fails).
Comment #17
analiam319@gmail.com CreditAttribution: analiam319@gmail.com commentedRe-made patch #9 for the latest 9.1.x (9.1.10 at the moment). I also added array access checks within the NegotationConfigureForm::configureForm where the reported errors were occurring.
Comment #21
smustgrave CreditAttribution: smustgrave at Mobomo commentedThis issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request as a guide.
This could use a test case to show the issue
Also tagging for accessibility review to make sure we are going down the right path.