FIX #408 Hide location row when no location @ashishxdev

[ashishxdev]

[gemini-code-assist]

Code Review

This pull request removes libc constraints from the package-lock.json and updates the location display logic in the Nunjucks templates. Feedback for src/_includes/bio.njk suggests simplifying the redundant conditional logic and addressing potential whitespace issues in the HTML output by using a more concise template structure and the trim filter for consistency.

[gemini-code-assist]

The logic for displaying the location has been made unnecessarily complex and introduces potential UI issues due to whitespace handling.

1. Redundancy: The nested if/elif checks for location are redundant because the entire block is already guarded by {% if location %}. The previous logic was more concise and readable.
2. Whitespace Sensitivity: In HTML, newlines and indentation inside a <span> are rendered as spaces. The current multi-line structure will likely result in unwanted leading and trailing whitespace around the location text.
3. Consistency: It is recommended to use | trim to ensure that strings containing only whitespace are correctly identified as empty, matching the logic applied in src/index.njk.

            {% if location and location | trim %}
            <div class="flex items-center gap-3">
              <span class="text-lg">📍</span>
              <span class="text-[var(--text-muted)]">{{ location }}{% if country %}, {{ country }}{% endif %}</span>
            </div>
            {% endif %}

[BaseMax]

Agreed, please fix it.