ashutoshkrris / TechyWrite-Old

A website to list down available paid writing opportunities for technical writers.
techywrite.vercel.app
MIT License
8 stars 11 forks source link

Responsive for Mobile screen. #46

Closed Rishav1707 closed 1 year ago

Rishav1707 commented 1 year ago

Description

Fixes #45

Type of change

Checklist:

Screenshots

Responsive

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
techywrite ✅ Ready (Inspect) Visit Preview Jan 8, 2023 at 4:12PM (UTC)
Rishav1707 commented 1 year ago

Hi @Rishav1707 Thank you for making the PR. But, issue #45 was related to just the pagination component, and the PR doesn't make any change to that.

Also in issue #45 it mention about responsive for mobile screen. I am only worked on responsiveness in mobile screen that's why I requested this PR.

ashutoshkrris commented 1 year ago

Hi @Rishav1707 Thank you for making the PR. But, issue #45 was related to just the pagination component, and the PR doesn't make any change to that.

Also in issue #45 it mention about responsive for mobile screen. I am only worked on responsiveness in mobile screen that's why I requested this PR.

That responsiveness was also related to pagination component actually.

Rishav1707 commented 1 year ago

Hi @Rishav1707 Thank you for making the PR. But, issue #45 was related to just the pagination component, and the PR doesn't make any change to that.

Also in issue #45 it mention about responsive for mobile screen. I am only worked on responsiveness in mobile screen that's why I requested this PR.

That responsiveness was also related to pagination component actually.

Ok I will try to solve it...

Rishav1707 commented 1 year ago

Hi @Rishav1707 Thank you for making the PR. But, issue #45 was related to just the pagination component, and the PR doesn't make any change to that.

Also in issue #45 it mention about responsive for mobile screen. I am only worked on responsiveness in mobile screen that's why I requested this PR.

That responsiveness was also related to pagination component actually.

Done @ashutoshkrris

ashutoshkrris commented 1 year ago

Did you delete the OpportunityList.vue file?

Rishav1707 commented 1 year ago

Did you delete the OpportunityList.vue file?

Yes, because I only want to commit style.css file

Rishav1707 commented 1 year ago

Did you delete the OpportunityList.vue file?

Sorry, by mistake it was deleted. But I recovered it...

Rishav1707 commented 1 year ago

@ashutoshkrris Please review my changes...

Rishav1707 commented 1 year ago

@ashutoshkrris Please review my changes...

?? @ashutoshkrris

Rishav1707 commented 1 year ago

Hi @Rishav1707 The mobile version still doesn't look as expected. Screenshot_2023-01-08-10-23-13-736_com android chrome

I think you have deployed the wrong link... Screenshot_2023-01-08-10-25-54-27_40deb401b9ffe8e1df2f1cc5ba480b12 See above it's quite better now for mobile screen

ashutoshkrris commented 1 year ago

Hi @Rishav1707 The mobile version still doesn't look as expected. Screenshot_2023-01-08-10-23-13-736_com android chrome

I think you have deployed the wrong link... Screenshot_2023-01-08-10-25-54-27_40deb401b9ffe8e1df2f1cc5ba480b12 See above it's quite better now for mobile screen

Could you try going to page number 4?

Rishav1707 commented 1 year ago

Hi @Rishav1707 The mobile version still doesn't look as expected. Screenshot_2023-01-08-10-23-13-736_com android chrome

I think you have deployed the wrong link... Screenshot_2023-01-08-10-25-54-27_40deb401b9ffe8e1df2f1cc5ba480b12 See above it's quite better now for mobile screen

Could you try going to page number 4?

Yaah now I see changes

Rishav1707 commented 1 year ago

Ok trying to fix this soon...

Rishav1707 commented 1 year ago

resp

How's this?

ashutoshkrris commented 1 year ago

Hi @Rishav1707 Since I changed the way pagination was working, there's no need to make any changes now in terms of responsiveness