responsive for dark and light mode , for all devices scroll bar added #141
Open
Khushee21 wants to merge 1 commit intoalvarotorrestx:devfrom
Open
responsive for dark and light mode , for all devices scroll bar added #141Khushee21 wants to merge 1 commit intoalvarotorrestx:devfrom
Khushee21 wants to merge 1 commit intoalvarotorrestx:devfrom
Conversation
alvarotorrestx
requested changes
Jun 22, 2025
Owner
alvarotorrestx
left a comment
alvarotorrestx
left a comment
There was a problem hiding this comment.
I like the style you created. There's just a lot of unnecessary additions and changes.
Please read each comment and adapt.
Thank you 😊
| "react-markdown": "^10.1.0", | ||
| "react-router-dom": "^6.23.1", | ||
| "remark-gfm": "^4.0.1", | ||
| "tailwind-scrollbar": "^4.0.2", |
Owner
There was a problem hiding this comment.
You installed this but you didn't use it. Please uninstall.
Comment on lines
+1304
to
+1309
| "node_modules/@types/prismjs": { | ||
| "version": "1.26.5", | ||
| "resolved": "https://registry.npmjs.org/@types/prismjs/-/prismjs-1.26.5.tgz", | ||
| "integrity": "sha512-AUZTa7hQ2KY5L7AmtSiqxlhWxb4ina0yd8hNbl4TWuqnv/pFP0nDMb3YrfSBf4hJVGLh2YEIBfKaBW/9UEl6IQ==", | ||
| "license": "MIT" | ||
| }, |
Owner
There was a problem hiding this comment.
The same goes for this, please uninstall.
Comment on lines
+1897
to
+1905
| "node_modules/clsx": { | ||
| "version": "2.1.1", | ||
| "resolved": "https://registry.npmjs.org/clsx/-/clsx-2.1.1.tgz", | ||
| "integrity": "sha512-eYm0QWBtUrBWZWG0d386OGAw16Z995PiOVo2B7bjWSbHedGl5e0ZWaq65kOGgUSNesEIDkB9ISbTg/JK9dhCZA==", | ||
| "license": "MIT", | ||
| "engines": { | ||
| "node": ">=6" | ||
| } | ||
| }, |
Owner
There was a problem hiding this comment.
The same goes for this, please uninstall.
Comment on lines
+5588
to
+5600
| "node_modules/prism-react-renderer": { | ||
| "version": "2.4.1", | ||
| "resolved": "https://registry.npmjs.org/prism-react-renderer/-/prism-react-renderer-2.4.1.tgz", | ||
| "integrity": "sha512-ey8Ls/+Di31eqzUxC46h8MksNuGx/n0AAC8uKpwFau4RPDYLuE3EXTp8N8G2vX2N7UC/+IXeNUnlWBGGcAG+Ig==", | ||
| "license": "MIT", | ||
| "dependencies": { | ||
| "@types/prismjs": "^1.26.0", | ||
| "clsx": "^2.0.0" | ||
| }, | ||
| "peerDependencies": { | ||
| "react": ">=16.0.0" | ||
| } | ||
| }, |
Owner
There was a problem hiding this comment.
The same goes for this, please uninstall.
Comment on lines
+6478
to
+6492
| "node_modules/tailwind-scrollbar": { | ||
| "version": "4.0.2", | ||
| "resolved": "https://registry.npmjs.org/tailwind-scrollbar/-/tailwind-scrollbar-4.0.2.tgz", | ||
| "integrity": "sha512-wAQiIxAPqk0MNTPptVe/xoyWi27y+NRGnTwvn4PQnbvB9kp8QUBiGl/wsfoVBHnQxTmhXJSNt9NHTmcz9EivFA==", | ||
| "license": "MIT", | ||
| "dependencies": { | ||
| "prism-react-renderer": "^2.4.1" | ||
| }, | ||
| "engines": { | ||
| "node": ">=12.13.0" | ||
| }, | ||
| "peerDependencies": { | ||
| "tailwindcss": "4.x" | ||
| } | ||
| }, |
Owner
There was a problem hiding this comment.
The same goes for this, please uninstall.
| @@ -1,5 +1,5 @@ | |||
| import axios from 'axios'; | |||
| const BASE_URL = 'http://localhost:5000'; | |||
| const BASE_URL = 'http://localhost:3000'; | |||
Owner
There was a problem hiding this comment.
This should remain port 5000.
This is the port our backend server uses.
Comment on lines
+51
to
+52
| <div className="h-screen overflow-y-auto scrollbar-thin scrollbar-thumb-base-content scrollbar-track-base-200 dark:scrollbar-thumb-base-100 dark:scrollbar-track-base-300"> | ||
| {/*responsive scroll bar added*/} |
Owner
There was a problem hiding this comment.
let's keep the scrollbar style changes just to main the scrollbar only, not overflow scrolling for now. so please remove.
|
|
||
| return ( | ||
| <div className="max-w-[90%] lg:max-w-4xl mx-auto p-6 bg-base-100 rounded-lg shadow-md"> | ||
| {/*Add scroll bar here if needed*/} |
| }, | ||
| plugins: [ | ||
| require("daisyui"), | ||
| require("tailwind-scrollbar"), |
Owner
There was a problem hiding this comment.
The same goes for this as earlier, please uninstall.
Comment on lines
+11
to
+16
| {/*you can add if needed for check res i have added this*/} | ||
| // app.get('/' , (req, res)=>{ | ||
| // res.send('Dev Connecct to server') | ||
| // console.log('Dev Coonect to server'); | ||
| // }) | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.