Skip to content

Fix text alignment issues on single listing page#2683

Open
RabbiIslamRony wants to merge 1 commit intosovware:developmentfrom
RabbiIslamRony:fix/single-listing-text-alignment
Open

Fix text alignment issues on single listing page#2683
RabbiIslamRony wants to merge 1 commit intosovware:developmentfrom
RabbiIslamRony:fix/single-listing-text-alignment

Conversation

@RabbiIslamRony
Copy link
Member

@RabbiIslamRony RabbiIslamRony commented Feb 8, 2026

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Security fix
  • Improvement
  • New Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Text changes
  • Other... Please describe:

Description

Fixed text alignment and vertical centering issues across several sections
of the single listing detail page.

Changes

Contact Info Section

  • Replaced margin-based alignment with inline-flex and align-items: center
    on .directorist-single-info__label__text and .directorist-single-info__value
  • Vertically centered the colon (:) separator using transform: translateY(-50%)

Details Content

  • Removed hardcoded #808080 color override on label icons
    (.directorist-single-info__label-icon .directorist-icon-mask:after)

Tag List

  • Fixed icon vertical misalignment by removing top: -5px offset
  • Added align-items: center to tag links for proper inline alignment
  • Scoped bottom margin to li:not(:last-child) to prevent extra spacing
    on the last item
  • Updated tag link color from --directorist-color-body to
    --directorist-color-dark and added hover color (--directorist-color-primary)

Any linked issues

Fixes #
https://taiga-sovware-u10698.vm.elestio.app/project/directorist/issue/2545

Checklist

@RabbiIslamRony RabbiIslamRony added this to the 8.5.10 milestone Feb 10, 2026
@Armanul46 Armanul46 modified the milestones: 8.5.10, v8.5.11 Feb 16, 2026
@iamibrahimriaz
Copy link
Collaborator

@RabbiIslamRony It works nicely when values are in one line. But multiline values breaks style.

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.

3 participants

Comments