Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Enhancement]: Better display of Templates instead of Carousel on Homepage #220

Open
hexronuspi opened this issue Dec 7, 2022 · 11 comments · May be fixed by #328
Open

[Enhancement]: Better display of Templates instead of Carousel on Homepage #220

hexronuspi opened this issue Dec 7, 2022 · 11 comments · May be fixed by #328
Assignees
@praptisharma28
Labels
iwoc24 medium

Comments

@hexronuspi
Copy link

hexronuspi commented Dec 7, 2022 •
edited
Loading

ss1

If the templates would have been boxed in a rectangle and placed there side by side then it would have looked much better.
Example :

test redo
@hexronuspi
Copy link
Author

@shelcia where is the main index.html file? I can't find it!

@shelcia
Copy link
Owner

shelcia commented Dec 7, 2022

it is a react project. The component resides in src/pages/auth/HomePage.jsx , TemplateCarousel is the component you will have to change

@shelcia
Copy link
Owner

shelcia commented Dec 18, 2022

@hexronuspi are you working on this ?

@hexronuspi
Copy link
Author

hexronuspi commented Dec 18, 2022 •
edited
Loading

No. I'm not familiar with react so I'm out! @shelcia

@shelcia shelcia added the iwoc label Feb 1, 2023
@shelcia shelcia changed the title Example templates structure format [Enhancement]: Better display of Templates instead of Carousel on Homepage Feb 11, 2023
@riyasinghsirohi
Copy link

heyy.. could you assign this task to me??

@shelcia shelcia added the iwoc24 label Jan 11, 2024
@JiyaGupta-cs
Copy link
Contributor

@shelcia Can you please assign this to me under IWOC?

@praptisharma28
Copy link
Contributor

Assign me this under IWOC @shelcia

@praptisharma28
Copy link
Contributor

Thankyou @shelcia , I am working on it, will give the PR soon!

praptisharma28 added a commit to praptisharma28/dynamic-portfolio that referenced this issue Jan 20, 2024
@praptisharma28
Enhancement : Carousel UI changed
38eab2d 
closes shelcia#220
@praptisharma28 praptisharma28 linked a pull request Jan 20, 2024 that will close this issue
Open
praptisharma28 added a commit to praptisharma28/dynamic-portfolio that referenced this issue Jan 24, 2024
@praptisharma28
home.scss path changed
fe6be86 
closes shelcia#220
@praptisharma28
Copy link
Contributor

@shelcia please change tag here in the issue to hard if you like, before merging the PR.

@hexronuspi
Copy link
Author

@praptisharma28
As someone who opened this issue around an year ago, this is not a hard task,

But the final decision is upto the owner of this repository.

@praptisharma28
Copy link
Contributor

ofcourse @hexronuspi , its upto @shelcia . Its just that it took a lot of time coz the template I used was a difficult one. But no issues any decision would be okay!

praptisharma28 added a commit to praptisharma28/dynamic-portfolio that referenced this issue Jan 26, 2024
@praptisharma28
corrections
50008aa 
closes shelcia#220
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@praptisharma28 praptisharma28

Labels
iwoc24 medium
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enhancement : Carousel UI changed praptisharma28/dynamic-portfolio
5 participants
@shelcia @riyasinghsirohi @hexronuspi @praptisharma28 @JiyaGupta-cs