Skip to content

ZA | 25-SDC-JULY | Luke Manyamazi | Sprint 1 | Extra long bloom #185

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Luke-Manyamazi wants to merge 3 commits into
base: main
Choose a base branch
from
+26 −4
Open

ZA | 25-SDC-JULY | Luke Manyamazi | Sprint 1 | Extra long bloom #185

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
9cc04b6
added code to limit bloom in the backend and control in the frontend
Luke-Manyamazi Apr 17, 2026
6c20c80
refactor: replaced hardcoded bloom character limit with constant
Luke-Manyamazi Apr 23, 2026
14f32a4
fixed the character limit constant reference
Luke-Manyamazi Apr 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions backend/data/blooms.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
from data.connection import db_cursor
from data.users import User

MAX_BLOOM_LENGTH = 280

@dataclass
class Bloom:
Expand All @@ -16,6 +17,9 @@ class Bloom:


def add_bloom(*, sender: User, content: str) -> Bloom:
if len(content) > MAX_BLOOM_LENGTH:
raise ValueError(f"Bloom must not exceed {MAX_BLOOM_LENGTH} characters")

hashtags = [word[1:] for word in content.split(" ") if word.startswith("#")]

now = datetime.datetime.now(tz=datetime.UTC)
Expand Down
26 changes: 22 additions & 4 deletions front-end/components/bloom.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,12 @@
* "sender": username,
* "content": "string from textarea",
* "sent_timestamp": "datetime as ISO 8601 formatted string"}

*/
const createBloom = (template, bloom) => {
if (!bloom) return;

const MAX_BLOOM_LENGTH = 280;
Copy link
Copy Markdown
Member

@illicitonion illicitonion Apr 24, 2026

Choose a reason for hiding this comment

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

There are still three occurrences of 280 in this file?


const bloomFrag = document.getElementById(template).content.cloneNode(true);
const bloomParser = new DOMParser();

Expand All @@ -21,14 +23,30 @@ const createBloom = (template, bloom) => {
const bloomTimeLink = bloomFrag.querySelector("a:has(> [data-time])");
const bloomContent = bloomFrag.querySelector("[data-content]");

const isTooLong = bloom.content && bloom.content.length > MAX_BLOOM_LENGTH;

// Highlight invalid blooms (UI only)
if (isTooLong) {
bloomArticle.style.border = "2px solid red";
bloomArticle.style.backgroundColor = "#fff5f5";
bloomArticle.title = `This bloom exceeds ${MAX_BLOOM_LENGTH} characters (invalid)`;
}

bloomArticle.setAttribute("data-bloom-id", bloom.id);
bloomUsername.setAttribute("href", `/profile/${bloom.sender}`);
bloomUsername.textContent = bloom.sender;
bloomTime.textContent = _formatTimestamp(bloom.sent_timestamp);
bloomTimeLink.setAttribute("href", `/bloom/${bloom.id}`);

const displayContent = isTooLong
? `[INVALID BLOOM - exceeds ${MAX_BLOOM_LENGTH} characters]\n\n${bloom.content}`
: bloom.content;

bloomContent.replaceChildren(
...bloomParser.parseFromString(_formatHashtags(bloom.content), "text/html")
.body.childNodes
...bloomParser.parseFromString(
_formatHashtags(displayContent),
"text/html"
).body.childNodes
);

return bloomFrag;
Expand Down Expand Up @@ -84,4 +102,4 @@ function _formatTimestamp(timestamp) {
}
}

export {createBloom};
export { createBloom };