Skip to content

feat(search bars): add clear button to forums, boards, people, network search (#42, #65)#116

Open
64johnlee wants to merge 11 commits into
RetroShare:masterfrom
64johnlee:ralph/RS-065c
Open

feat(search bars): add clear button to forums, boards, people, network search (#42, #65)#116
64johnlee wants to merge 11 commits into
RetroShare:masterfrom
64johnlee:ralph/RS-065c

Conversation

@64johnlee
Copy link
Copy Markdown

@64johnlee 64johnlee commented May 3, 2026

Summary

Add X button to clear search text in multiple sections.

Chat (#42)

  • Emoji button next to send input (😊 inserts random emoji)
  • Emoji keyboard support on mobile (inputmode="emoji")
  • Improved user list in right sidebar (compact rows, smaller font)

Search Clear Buttons (#65)

  • Forums: Clear button in SearchBar
  • Boards: Clear button in SearchBar
  • People: Clear button in contacts SearchBar
  • Network: Clear button in FriendsList SearchBar

Files Changed

  • webui-src/app/chat/chat.js - Emoji button + input improvements
  • webui-src/app/forums/forums_util.js - SearchBar with clear button
  • webui-src/app/boards/boards_util.js - SearchBar with clear button
  • webui-src/app/people/people_util.js - SearchBar with clear button
  • webui-src/app/network/network.js - SearchBar with clear button
  • webui-src/styles.css - Chat + search CSS

@64johnlee 64johnlee changed the title style(chat): improve user list display in right sidebar (#42) feat(chat): emoji button + user list improvements (#42) May 3, 2026
@64johnlee 64johnlee changed the title feat(chat): emoji button + user list improvements (#42) feat(chat,forums): emoji button + search clear + user list improvements (#42, #65) May 3, 2026
@64johnlee 64johnlee changed the title feat(chat,forums): emoji button + search clear + user list improvements (#42, #65) feat(search bars): add clear button to forums, boards, people, network search (#42, #65) May 3, 2026
@defnax
Copy link
Copy Markdown
Contributor

defnax commented May 24, 2026

can you clean all your codes`? 9 changes but 54 commits and a agents.md for what you need this bump stuff? all your prs has this bad thing

@64johnlee
Copy link
Copy Markdown
Author

Thanks for the feedback. Cleaned up the branch — removed all the bump commits (now 11 commits matching the actual changes) and removed the stray AGENTS.md file.

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