-
Notifications
You must be signed in to change notification settings - Fork 101
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.
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
#12 issue solved - Changes done in the UI of home page better user Interaction. #170
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
π Thank you @SoumyaU25 for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay @Harshdev098 I'll do that |
Hello @Harshdev098 I have done the changes , I have added a Search bar also. Please Check it out. |
@Harshdev098 actually it's working fine at my end, I have set the right path for it. |
public/css/main_page_style.css
Outdated
.banner { | ||
width: 100%; | ||
height: 100vh; | ||
background-image: linear-gradient(rgba(8, 23, 82, 0.75), rgba(30, 2, 30, 0.75)), url('public/images/calm.jpg'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
edit the url with this ../images/calm.jpg
<div class="nav"> | ||
<img class="logo" src="images/logo.png" alt=""> | ||
<form class="search-bar"> | ||
<input type="text" placeholder="Search..."> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Increase the input size of search box something like 20vw
<li><a href="">About</a></li> | ||
<li><a href="">Add Topic</a></li> | ||
<li onmouseover="show()" onmouseout="hide()"><a href="" | ||
title="View Notifications"><i id="firsticon" class="fa-sharp fa-regular fa-bell" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The notification btn is not working as expected onmouseover and onmouseout properties
@Harshdev098 I have done all the changes as you said |
@SoumyaU25 Delete the .vscode folder |
Okay @Harshdev098 |
Done @Harshdev098 |
@SoumyaU25 Can you update your PR description referencing the issue number you have fixed |
π Description
π¨ Changes Made
β Checklist
Before submitting the PR, please make sure you have completed the following:
π·οΈ Types of Changes
What type of changes does your code introduce? (Check all that apply)
Bug fix (non-breaking change which fixes an issue) π
New feature (non-breaking change which adds functionality) β¨
UI enhancement (non-breaking change which enhances UI) π¨
Documentation update π
Fixes #
πΈ Screenshots (if applicable)
Thank you for contributing to Research Nexus! We look forward to reviewing your PR and getting it merged!