8000 $additionalFieldset anchor navigation is not displayed in form · Issue #2650 · area17/twill · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

$additionalFieldset anchor navigation 8000 is not displayed in form #2650

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.

Sign up for GitHub

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

Jump to bottom
Closed
HarryThe3rd opened this issue Sep 12, 2024 · 0 comments · Fixed by #2651
Closed

$additionalFieldset anchor navigation is not displayed in form #2650

HarryThe3rd opened this issue Sep 12, 2024 · 0 comments · Fixed by #2651

Comments

@HarryThe3rd
Copy link
Contributor

Description

In the form.blade.php for a model, I defined several additionalFieldsets but the anchor navigation is not displayed.

Steps to reproduce

Go to a form.blade.php define some addtional fieldsets like

@extends('twill::layouts.form', [
    'disableContentFieldset' => true,
    'additionalFieldsets' => [
        ['fieldset' => 'baseinfo', 'label' => twillTrans('modules/programs.baseinfo')],
        ['fieldset' => 'structure', 'label' => twillTrans('modules/programs.structure')],
        ['fieldset' => 'contents', 'label' => twillTrans('modules/programs.contents')],
        ['fieldset' => 'seo', 'label' => 'SEO'],
      ],
])

then wrapped the formfields inside the @section('fieldsets') with

@formFieldset(['id' => 'baseinfo', 'title' => twillTrans('modules/programs.baseinfo'), 'open' => true])
    ...
@endformFieldset

Expected result

An anchor navigation should be shown in form of the model.

Actual result

The anchor navigation is not shown.

Versions

Twill version: 3.3.1
Laravel version: 11.22.0
PHP version: 8.3.9
Database engine: MySQL 8.0.27
HarryThe3rd added a commit to HarryThe3rd/twill that referenced this issue Sep 12, 2024
@HarryThe3rd
fix area17#2650: Added parentheses to fix the order of evaluation bet…
4ec829f 
…ween the null coalescing operator (??) and the ternary operator (?:)
@HarryThe3rd HarryThe3rd mentioned this issue Sep 12, 2024
Merged
ifox added a commit to HarryThe3rd/twill that referenced this issue Oct 14, 2024
@ifox
Merge branch '3.x' into fix/area17#2650-additional-fieldset-anchor-na…
dc81279 
…vigation
ifox pushed a commit that referenced this issue Oct 14, 2024
@HarryThe3rd @ifox
fix #2650: Added parentheses to fix the order of evaluation between t…
58dc470 
…he null coalescing operator (??) and the ternary operator (?:)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix #2650: Added parentheses to fix the order of evaluation between t… HarryThe3rd/twill
1 participant
@HarryThe3rd
0