Skip to content

7840: Improved handling of empty step title#395

Merged
rimi-itk merged 2 commits into
mainfrom
feature/7840-step-title
Jun 29, 2026
Merged

7840: Improved handling of empty step title#395
rimi-itk merged 2 commits into
mainfrom
feature/7840-step-title

Conversation

@rimi-itk

@rimi-itk rimi-itk commented Jun 29, 2026

Copy link
Copy Markdown

Link to ticket

https://leantime.itkdev.dk/_#/tickets/showTicket/7840

Description

Removed hack trying to work around empty title on a step. The hack made the step content appear twice.

Screenshot of the result

Before:

Screenshot 2026-06-29 at 11 27 32 Screenshot 2026-06-29 at 11 27 29

Notice duplicated content (“Når der er behov for at …”).

After:

Screenshot 2026-06-29 at 11 29 10 Screenshot 2026-06-29 at 11 29 06

If your change affects the user interface you should include a screenshot of the result with the pull request.

Checklist

  • My code passes our test (all our tests).
  • My code passes our static analysis suite.
  • My code passes our continuous integration process.

If your code does not pass all the requirements on the checklist you have to add a comment explaining why this change
should be exempt from the list.

Additional comments or questions

If you have any further comments or questions for the reviewer please add them here.

@rimi-itk rimi-itk force-pushed the feature/7840-step-title branch from 92c227f to 34661c5 Compare June 29, 2026 09:32
@rimi-itk rimi-itk marked this pull request as ready for review June 29, 2026 09:32
@rimi-itk rimi-itk requested a review from martinyde June 29, 2026 09:32
@rimi-itk rimi-itk merged commit 0004221 into main Jun 29, 2026
14 of 15 checks passed
@rimi-itk rimi-itk deleted the feature/7840-step-title branch June 29, 2026 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants