Skip to content

Commit

Permalink
Bugfix/issue 435 (#437)
Browse files Browse the repository at this point in the history
* fixed the focus rings on navbuttons

* fixed the focus rings on navbuttons file
  • Loading branch information
AhmedHHamdy authored Jul 25, 2023
1 parent 760cfe9 commit 7534047
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions client/src/features/home/NavButtons.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const NavButtons = ({ HomeIcon, LoginIcon, CalendarIcon }) => {
{/* Home Button */}
<button
onClick={() => routing.setCurrentPage("landingPage")}
className="bg-white border-2 border-black rounded-2xl flex flex-col justify-center items-center p-1 w-24 h-24 lg:w-28 lg:h-28 xl:w-32 xl:h-32"
className="bg-white border-2 border-black rounded-2xl flex flex-col justify-center items-center p-1 w-24 h-24 lg:w-28 lg:h-28 xl:w-32 xl:h-32 focus:ring-4 focus:ring-black"
>
<div className="flex justify-center">
<HomeIcon className="text-mainBlue w-7 h-7 lg:w-10 lg:h-10" />
Expand All @@ -28,7 +28,7 @@ const NavButtons = ({ HomeIcon, LoginIcon, CalendarIcon }) => {
</button>
{/* Log In & Log Out Button */}
<button
className="bg-white border-2 border-black rounded-2xl flex flex-col justify-center items-center p-1 w-24 h-24 lg:w-28 lg:h-28 xl:w-32 xl:h-32"
className="bg-white border-2 border-black rounded-2xl flex flex-col justify-center items-center p-1 w-24 h-24 lg:w-28 lg:h-28 xl:w-32 xl:h-32 focus:ring-4 focus:ring-black"
onClick={isAuthenticated() ? logout : handleLogin}
>
<div className="flex justify-center">
Expand All @@ -42,7 +42,7 @@ const NavButtons = ({ HomeIcon, LoginIcon, CalendarIcon }) => {

<button
onClick={() => navigate("calendar")}
className="bg-white border-2 border-black rounded-2xl flex flex-col justify-center items-center p-1 w-24 h-24 lg:w-28 lg:h-28 xl:w-32 xl:h-32"
className="bg-white border-2 border-black rounded-2xl flex flex-col justify-center items-center p-1 w-24 h-24 lg:w-28 lg:h-28 xl:w-32 xl:h-32 focus:ring-4 focus:ring-black"
>
<div className="flex justify-center">
<CalendarIcon className="text-mainOrange w-7 h-7 lg:w-10 lg:h-10" />
Expand Down

0 comments on commit 7534047

Please sign in to comment.