feat(web): form exec

[gfanton]

TODO

[Gno2D2]

🛠 PR Checks Summary

🔴 Changes related to gnoweb must be reviewed by its codeowners

Manual Checks (for Reviewers):

• IGNORE the bot requirements for this PR (force green CI check)

Read More

🤖 This bot helps streamline PR reviews by verifying automated checks and providing guidance for contributors and reviewers.

✅ Automated Checks (for Contributors):

🟢 Maintainers must be able to edit this pull request (more info)
🔴 Changes related to gnoweb must be reviewed by its codeowners

☑️ Contributor Actions:

1. Fix any issues flagged by automated checks.
2. Follow the Contributor Checklist to ensure your PR is ready for review.
   • Add new tests, or document why they are unnecessary.
   • Provide clear examples/screenshots, if necessary.
   • Update documentation, if required.
   • Ensure no breaking changes, or include BREAKING CHANGE notes.
   • Link related issues/PRs, where applicable.

☑️ Reviewer Actions:

1. Complete manual checks for the PR, including the guidelines and additional checks if applicable.

📚 Resources:

• Report a bug with the bot.
• View the bot’s configuration file.

Debug

Automated Checks

Maintainers must be able to edit this pull request (more info)

If

🟢 Condition met
└── 🟢 And
    ├── 🟢 The base branch matches this pattern: ^master$
    └── 🟢 The pull request was created from a fork (head branch repo: gfanton/gno)

Then

🟢 Requirement satisfied
└── 🟢 Maintainer can modify this pull request

Changes related to gnoweb must be reviewed by its codeowners

If

🟢 Condition met
└── 🟢 And
    ├── 🟢 The base branch matches this pattern: ^master$
    └── 🟢 A changed file matches this pattern: ^gno.land/pkg/gnoweb/ (filename: gno.land/pkg/gnoweb/app.go)

Then

🔴 Requirement not satisfied
└── 🔴 Or
    ├── 🔴 Or
    │   ├── 🔴 And
    │   │   ├── 🔴 Pull request author is user: alexiscolin
    │   │   └── 🔴 This user reviewed pull request: gfanton (with state "APPROVED")
    │   └── 🔴 And
    │       ├── 🟢 Pull request author is user: gfanton
    │       └── 🔴 This user reviewed pull request: alexiscolin (with state "APPROVED")
    └── 🔴 And
        ├── 🟢 Not (🔴 Pull request author is user: alexiscolin)
        ├── 🔴 Not (🟢 Pull request author is user: gfanton)
        └── 🔴 Or
            ├── 🔴 This user reviewed pull request: alexiscolin (with state "APPROVED")
            └── 🔴 This user reviewed pull request: gfanton (with state "APPROVED")

Manual Checks

**IGNORE** the bot requirements for this PR (force green CI check)

If

🟢 Condition met
└── 🟢 On every pull request

Can be checked by

• Any user with comment edit permission

[codecov]

Codecov Report

❌ Patch coverage is 75.62380% with 127 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
gno.land/pkg/gnoweb/markdown/ext_forms.go	86.27%	49 Missing and 3 partials ⚠️
gno.land/pkg/gnoweb/client_mock.go	0.00%	22 Missing ⚠️
gno.land/pkg/gnoweb/render.go	33.33%	19 Missing and 1 partial ⚠️
gno.land/pkg/gnoweb/client.go	0.00%	13 Missing ⚠️
gno.land/pkg/gnoweb/components/template.go	0.00%	9 Missing and 1 partial ⚠️
gno.land/pkg/gnoweb/markdown/context.go	60.00%	4 Missing and 4 partials ⚠️
gno.land/pkg/gnoweb/components/ui_breadcrumb.go	0.00%	1 Missing ⚠️
gno.land/pkg/gnoweb/handler_http.go	93.75%	0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

[michelleellen]

@alexiscolin @aeddi any way we can get this merged? It would be great to have for Boards given currently it is challenging to fill in any long form content for general users.

[alexiscolin]

@michelleellen we are currently working on it together. We will have some news very soon.

[aeddi]

Suggested change

	background-color: var(--transparent);
	background-color: var(--g-color-transparent);

@alexiscolin Wouldn't it be ⁠--g-color-transparent instead like the rest of the file?

[aeddi]

There was a tricky issue to solve around generating command blocks from the markdown package (form exec) and the components package (actions page). For the sake of consistency and to fix the gosec-related problems, I made the command block HTML be generated via a template instead of a function using printf.

In this scenario, because the markdown package needed to import the command block template and the associated types, I had to fix a circular import by moving the TocItem definition into a subpackage of markdown. It would probably be cleaner to move all of these types into a package shared by markdown and components. While we wait for @gfanton to get back, I propose merging this as-is to unblock @jeronimoalbi, and letting @gfanton post-review or make changes (if needed) when he return.

The two commits on this are: 9ee5494 and bfba98f.