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.

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

[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

Comments

@hexronuspi
Copy link

hexronuspi commented Dec 7, 2022

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

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??

@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 praptisharma28 linked a pull request Jan 20, 2024 that will close this issue
praptisharma28 added a commit to praptisharma28/dynamic-portfolio that referenced this issue Jan 24, 2024
@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants