refactor: replace paragraph with span to avoid alignment breaking

[paodb]

Close #12

Summary by CodeRabbit

• Style
  • Polished the monthly schedule label presentation for improved visual consistency and alignment across themes.
  • Reduced unnecessary block-level spacing, making the control slightly more compact and visually integrated with surrounding elements.
  • Minor UI refinement only; no changes to behavior, configuration, or user workflows.

[coderabbitai]

Walkthrough

Replaced the monthly layout’s suffix component from Paragraph to Span for the month label in CronExpressionField.java, updating imports accordingly. No other logic or behavior changes are included.

Changes

Cohort / File(s)	Summary
Monthly layout label element swap src/main/java/com/flowingcode/vaadin/addons/cronexpressionfield/CronExpressionField.java	Replaced Paragraph with Span for the month label suffix component; removed Paragraph import and added/used Span.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title succinctly and accurately describes the main change by stating that a paragraph element was replaced with a span to fix alignment issues, matching the refactoring performed in the code.
Linked Issues Check	✅ Passed	The pull request addresses the alignment issue described in issue #12 by replacing the paragraph wrapper with a span in the monthly layout, directly targeting the misalignment of the “Repeat every” control and fulfilling the expected outcome of aligned content.
Out of Scope Changes Check	✅ Passed	All modifications are confined to the replacement of a paragraph element with a span in the CronExpressionField component; no unrelated or additional functionality changes are introduced.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch issue-12

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b323b31 and 6037e71.

📒 Files selected for processing (1)

• src/main/java/com/flowingcode/vaadin/addons/cronexpressionfield/CronExpressionField.java (1 hunks)

🔇 Additional comments (1)

src/main/java/com/flowingcode/vaadin/addons/cronexpressionfield/CronExpressionField.java (1)

343-343: LGTM! Correct fix for the alignment issue.

Replacing Paragraph with Span is the appropriate solution. Paragraph is a block-level element that would break inline alignment when used as a suffix component, while Span is an inline element that properly aligns with the adjacent input field and buttons in the "Repeat every" control.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}