Skip to content

Add get_horizontal_link_block to return the horizontal link block for…#16190

Merged
adalcima merged 2 commits into
feature/horizontal-link-block-implementationfrom
TP1-4042-Add-method-for-horizontal-link-block-inheritance-logic
Jul 13, 2026
Merged

Add get_horizontal_link_block to return the horizontal link block for…#16190
adalcima merged 2 commits into
feature/horizontal-link-block-implementationfrom
TP1-4042-Add-method-for-horizontal-link-block-inheritance-logic

Conversation

@adalcima

@adalcima adalcima commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Description

Prepares GeneralPage model to support an inherited horizontal link block snippet.
Related issue: TP1-4042

Main changes:

  • Added get_horizontal_link_block method
    • Returns the page's own snippet if assigned; otherwise queries ancestor GeneralPage nodes
    • Result is cached on the instance to avoid repeated queries per request
  • Added TODO placeholder for the missing horizontal_link_block FK field (to be implemented in this ticket TP1-4041)

Still pending (not in this PR)

  • HorizontalNavBar snippet model (just a suggested name)
  • FK field + migration
  • Template integration: {% with nav=page.get_horizontal_link_block %}...{% endwith %}

@adalcima
adalcima requested a review from ramram-mf July 9, 2026 17:04
@adalcima
adalcima marked this pull request as ready for review July 9, 2026 17:16
@mergify

mergify Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@ramram-mf ramram-mf left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM lets merge it to the feature branch and keep shaping this.

@adalcima
adalcima merged commit 24a7ca4 into feature/horizontal-link-block-implementation Jul 13, 2026
10 of 11 checks passed
@adalcima
adalcima deleted the TP1-4042-Add-method-for-horizontal-link-block-inheritance-logic branch July 13, 2026 16:01
ramram-mf added a commit that referenced this pull request Jul 16, 2026
* Add get_horizontal_link_block to return the horizontal link block for… (#16190)

* Add get_horizontal_link_block to return the horizontal link block for General page

* Fix linting issues

* TP1-4041 Implement horizontal link block (#16204)

* TP1-4041 Implement horizontal link block

* TP1-4041 Fix horizontal link block formatting

* TP1-4041 Refine horizontal link interactions

* TP1-4041 Address horizontal link design feedback

* TP1-4041 Extend horizontal divider through menu padding

* TP1-4041 Use tight horizontal divider artwork

---------

Co-authored-by: adalcima <cima.adalberto@gmail.com>
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