search

SpecialProjectADU / Site

1 stars 0 forks source link

Redesigned the main page #48

Closed AlyafeiAli closed 1 year ago

AlyafeiAli commented 1 year ago

Actually @AlyafeiAli , the first 4 lines of Navbar.css are using the hover quesdo class selector, and you can't add hover effect with bootstrap. I am sorry for that didn't see it on my first look. I can be wrong sometimes, its good to push back against my arguments.

Your pull request is approved. Great job

I really did think it was related to the work I was doing offline lol
And yes, absolutely @binHarby , I agree. I'll push another commit to replace BrowseVideo's navbar and to remove the events section in index.html (as per Moutasim's request).
Thank you for your work.

AlyafeiAli commented 1 year ago

Just noticed @Moutasim02 's branches conflicts... I'll wait till they get resolved, and then I'll replace the navbar.