The home icon / text [VIdeoCutTool] should be clickable, and should redirect to /
Description
Details
| Title | Reference | Author | Source Branch | Dest Branch | |
|---|---|---|---|---|---|
| videocuttool: Made the logo and icon and title clickabe. ... | cloudvps-repos/videocuttool/VideoCutTool!47 | aditya0545 | clickable-icon | master | |
| videocuttool: Made the logo and icon and title clickabe. ... | cloudvps-repos/videocuttool/VideoCutTool!46 | aditya0545 | clickable-icon | master | |
| videocuttool: Made the logo and icon and title clickabe. ... | cloudvps-repos/videocuttool/VideoCutTool!32 | aditya0545 | clickable-icon | master |
Related Objects
- Mentioned In
- T331954: Add a Back-To-home button on result page
Event Timeline
Thank you for tagging this task with good first task for Wikimedia newcomers!
Newcomers often may not be aware of things that may seem obvious to seasoned contributors, so please take a moment to reflect on how this task might look to somebody who has never contributed to Wikimedia projects.
A good first task is a self-contained, non-controversial task with a clear approach. It should be well-described with pointers to help a completely new contributor, for example it should clearly pointed to the codebase URL and provide clear steps to help a contributor get setup for success. We've included some guidelines at https://phabricator.wikimedia.org/tag/good_first_task/ !
Thank you for helping us drive new contributions to our projects <3
I have understand the problem and I am going to do changes in this file
VideoCutTool/src/components/Header.jsx
am i going right ?
hey @Adityakumar0545 yes you're going in the right direction.
may i add a couple of check points for exit criteria here
- functionality: redirect to /
- check on both web and msite (mobile view)
- add a unit test mimicking the same functionality (you can refer to home.test.jsx for reference) [ probably something around fireEvent.click(); should work ]
@Reputation22 Thanks, I will do same
@RoadToWiki is My community tags, I want to add all the works in my community tags.
I have submitted
https://gitlab.wikimedia.org/cloudvps-repos/videocuttool/VideoCutTool/-/merge_requests/32
hi @Adityakumar0545 i've raised some comments here https://gitlab.wikimedia.org/cloudvps-repos/videocuttool/VideoCutTool/-/merge_requests/32
lmk when you resolve them
Since there was no response from the assignee, removing for now so that others can take it up.
Hi @Reputation22,
Thank you for your feedback on my previous PR. I’ve made the suggested changes and updated this PR accordingly. Could you please review it? If there’s still anything that needs improvement, I’d really appreciate your guidance, as I’m very eager to contribute and learn from this project.
https://gitlab.wikimedia.org/cloudvps-repos/videocuttool/VideoCutTool/-/merge_requests/46
hey @Adityakumar0545 have reviewed the PR and have left some comments. please check and address
thanks
HI, @Reputation22
Thanks for Review
can you check again this request
https://gitlab.wikimedia.org/cloudvps-repos/videocuttool/VideoCutTool/-/merge_requests/47
hey @Adityakumar0545 sorry for the late response, have reviewed your MR and left some comments
please check and resolve them
also feel free to discuss your open pointers/doubts regarding the ticket here (if any)
rishabh23 opened https://gitlab.wikimedia.org/cloudvps-repos/videocuttool/VideoCutTool/-/merge_requests/63
T387497: make icon text clickable
Hi @Reputation22 @Aklapper ,
I created the MR for this issue. I wasn't able to test it after login due to IP block issue (#T424174).
Please let me know your thoughts on this.
https://gitlab.wikimedia.org/cloudvps-repos/videocuttool/VideoCutTool/-/merge_requests/63

