Fix #86: Investigate chat system rendering bug observed while away from dev environment #118

Open

typosaurus wants to merge 2 commits from typosaurus/ticket-86 into master

typosaurus commented

2026-07-23 03:32:02 +02:00

Collaborator

Resolves #86.

Plan

Plan

Edit devplacepy/static/css/messages.css.

Locate the existing message-bubble style rules (around lines 259-281).
Append the following rule after the .message-bubble.mine block, ensuring it scopes links inside outgoing bubbles:

/* Override global and .rendered-content link colours for outgoing bubbles
   to prevent orange-on-orange invisibility (accent background). */
.message-bubble.mine a,
.message-bubble.mine .rendered-content a {
    color: var(--white);
    text-decoration: underline;
}

Run the project test command to confirm no regressions:
```
pip install -e '.[dev]' -q && python -m pytest tests/ tests/api/ tests/e2e/ tests/unit/
```
(If the test_command above is expected to be run without arguments, adjust accordingly; the full suite is safe as the change is purely CSS.)
Optionally add a brief client-side Playwright test that sends a message with a link and asserts the computed link colour inside .message-bubble.mine is rgb(255, 255, 255) (white). This is not strictly required because the fix is CSS-only, but it closes the coverage gap identified in the investigation.

Definition of done

CSS rule added in devplacepy/static/css/messages.css that sets color: var(--white) on <a> elements inside .message-bubble.mine.
Running pip install -e '.[dev]' -q && python -m pytest tests/ tests/api/ tests/e2e/ tests/unit/ exits with code 0 (all tests pass).
No new lint violations introduced (verify with ruff check --fix or project’s lint command if one exists).
Manual confirmation (or Playwright assertion) that links inside outgoing chat bubbles render as white-on-orange.

Opened automatically by Typosaurus.

Resolves #86. ## Plan **Plan** 1. **Edit** `devplacepy/static/css/messages.css`. - Locate the existing message-bubble style rules (around lines 259-281). - Append the following rule **after** the `.message-bubble.mine` block, ensuring it scopes links inside outgoing bubbles: ```css /* Override global and .rendered-content link colours for outgoing bubbles to prevent orange-on-orange invisibility (accent background). */ .message-bubble.mine a, .message-bubble.mine .rendered-content a { color: var(--white); text-decoration: underline; } ``` 2. **Run the project test command** to confirm no regressions: ```bash pip install -e '.[dev]' -q && python -m pytest tests/ tests/api/ tests/e2e/ tests/unit/ ``` (If the `test_command` above is expected to be run without arguments, adjust accordingly; the full suite is safe as the change is purely CSS.) 3. **Optionally** add a brief client-side Playwright test that sends a message with a link and asserts the computed link colour inside `.message-bubble.mine` is `rgb(255, 255, 255)` (white). This is not strictly required because the fix is CSS-only, but it closes the coverage gap identified in the investigation. --- **Definition of done** - [x] CSS rule added in `devplacepy/static/css/messages.css` that sets `color: var(--white)` on `<a>` elements inside `.message-bubble.mine`. - [x] Running `pip install -e '.[dev]' -q && python -m pytest tests/ tests/api/ tests/e2e/ tests/unit/` exits with code 0 (all tests pass). - [x] No new lint violations introduced (verify with `ruff check --fix` or project’s lint command if one exists). - [x] Manual confirmation (or Playwright assertion) that links inside outgoing chat bubbles render as white-on-orange. Opened automatically by Typosaurus.

typosaurus added 2 commits 2026-07-23 03:32:03 +02:00