Skip to content

AI-175 Fix favicon and prevent chatbot text overflow#66

Merged
DavidH-1 merged 2 commits intoopenMF:devfrom
iramsk02:fix/favicon-and-chatbot-overflow
Mar 11, 2026
Merged

AI-175 Fix favicon and prevent chatbot text overflow#66
DavidH-1 merged 2 commits intoopenMF:devfrom
iramsk02:fix/favicon-and-chatbot-overflow

Conversation

@iramsk02
Copy link
Copy Markdown
Contributor

@iramsk02 iramsk02 commented Jan 2, 2026

Summary

This PR addresses two UI issues in the Next.js application:

  • Replaces the default Vercel favicon with the Mifos logo
  • Fixes chatbot response text overflowing its container

Related Jira Ticket

(https://mifosforge.jira.com/browse/AI-175)

Changes Made

  • Added icon.png to the app root directory to override the default Vercel favicon
  • Updated chatbot styles to ensure proper text wrapping and overflow handling

Testing

  • Tested locally in development mode
  • Verified favicon update in the browser tab
  • Confirmed chatbot responses remain within the container

@staru09
Copy link
Copy Markdown
Member

staru09 commented Jan 20, 2026

Have you tested this PR?
Please also give a snippet of the UI about what has changed.
A screenshot of before and after would be cool.

@iramsk02
Copy link
Copy Markdown
Contributor Author

  • Text Overflow before
Screenshot (469)
  • After
Screenshot 2026-01-02 100555
  • Favicon before
Screenshot 2026-01-02 100419
  • After Mifos Icon
Screenshot 2026-01-02 100503 image

@staru09
Copy link
Copy Markdown
Member

staru09 commented Jan 26, 2026

lgtm
@DavidH-1 please merge this one

@DavidH-1
Copy link
Copy Markdown
Collaborator

needs to be against Dev not main

@DavidH-1
Copy link
Copy Markdown
Collaborator

please put Jira ticket ref in title

@DavidH-1
Copy link
Copy Markdown
Collaborator

CLA Check = FAILED please sign contributor license agreement

@iramsk02 iramsk02 changed the title Fix favicon and prevent chatbot text overflow AI-175 Fix favicon and prevent chatbot text overflow Mar 11, 2026
@iramsk02 iramsk02 changed the base branch from main to dev March 11, 2026 16:56
@iramsk02 iramsk02 requested a review from a team March 11, 2026 16:56
@iramsk02
Copy link
Copy Markdown
Contributor Author

@DavidH-1

I've fulfilled the above mentioned requirements

  • Sign Contributor Licence Agreement
  • Changed Base branch Main to dev
  • updated title to include jira ticket

@DavidH-1
Copy link
Copy Markdown
Collaborator

CLA check = passed

@DavidH-1
Copy link
Copy Markdown
Collaborator

Its showing a conflict before I can merge.

@iramsk02
Copy link
Copy Markdown
Contributor Author

@DavidH-1 I've resolved the conflicts

@DavidH-1 DavidH-1 merged commit 56d25fb into openMF:dev Mar 11, 2026
1 check passed
@iramsk02 iramsk02 deleted the fix/favicon-and-chatbot-overflow branch March 12, 2026 05:03
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