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

[Bug] Button alignment hardcoded with pixel values #143

Closed
1 task done
Lakshay-Pruthi opened this issue Apr 1, 2023 · 10 comments · Fixed by #304
Closed
1 task done

[Bug] Button alignment hardcoded with pixel values #143

Lakshay-Pruthi opened this issue Apr 1, 2023 · 10 comments · Fixed by #304
Assignees
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents 🏁 status: ready for work Ready for work

Comments

@Lakshay-Pruthi
Copy link
Contributor

Lakshay-Pruthi commented Apr 1, 2023

Description

The button alignment on the resource page is done using hardcode pixels which is not a good practice. Also it makes the responsiveness dependent on the dimension of buttons. Any changes made to button dimension will also require changes here. Example --> Missing repo button caused responsiveness issues.
image

Unnecessary code which can be deleted
image
Instead of this we can use flex properties in style.css

Reproduction

  1. Go to resource page
  2. See error.

Expectation

It should be properly aligned in the centre.

Resolution

  • I would be interested in resolving this bug.
@Lakshay-Pruthi Lakshay-Pruthi added 💻 aspect: code Concerns the software code in the repository 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents labels Apr 1, 2023
@Xaid-vfx
Copy link
Contributor

Xaid-vfx commented Apr 1, 2023

Good catch. Apparently, the misalignment of download button is only in open data handbook resource and no other.
The misalignment is not due hard coded pixels because if it were it would have been same for all resources.

Thanks to your issue, I had to take a look at it. The reason behind this is a commit from @Keerthivardhan1 . Apparently, he was removed all repo links from resources that were not working. But due to this change in resource.html , the repo link of Open data handbook which was working fine is also compromised. So next to the download button there should be a repo button link, which is missing.

Here's the repo url that was compromised.

If this was intentional, then I would suggest to just remove the repourl from opendatahandbook.html and the issue would be resolved.

I request @possumbilities to take a look into this as the change is live already.

Raised an issue #144 regarding this. @Lakshay-Pruthi, you are welcome to take it up.

@Lakshay-Pruthi
Copy link
Contributor Author

Lakshay-Pruthi commented Apr 1, 2023

@Xaid-vfx Thanks for the details you provided. I completely agree the misalignment is due to missing repo button but ultimately this needs to be corrected as it is not a good practice and any changes made to dimensions of download or repo button will result into misalignment in future.

@Lakshay-Pruthi Lakshay-Pruthi changed the title [Bug] Misalignment of download button due to hard coded pixels [Bug] Button alignment hardcoded with pixel values Apr 1, 2023
@Lakshay-Pruthi
Copy link
Contributor Author

Lakshay-Pruthi commented Apr 1, 2023

Updated issue title and description to address the main point.

@Keerthivardhan1
Copy link
Contributor

@Lakshay-Pruthi
under the guidance of @possumbilities I had successfully resolved all the broken links this is one from them.
I think this issue is occurring with only --> open data hand book page.

Clicking on the button previously led to a website that is no longer available on the internet, namely lawrent.com.
we should not include the repository button again. For additional information, please refer to #11.

@Keerthivardhan1
Copy link
Contributor

@Lakshay-Pruthi, you only need to organize the CSS styles, and your exploration in the website is really appreciative.

@Xaid-vfx
Copy link
Contributor

Xaid-vfx commented Apr 1, 2023

Hey @Lakshay-Pruthi and @Keerthivardhan1,
We can carry this conversation to #144, as it addresses the issue accurately.

@abhishekshah5486
Copy link

abhishekshah5486 commented Aug 4, 2024

Please assign this issue to me.
I have setup the project locally, would like to work upon this issue.

@Murdock9803
Copy link
Member

@abhishekshah5486 The UI for the page has major improvements done, and the issue has become irrelevant since then.
We can close the issue as completed @possumbilities 🙌

@possumbilities
Copy link
Contributor

@Murdock9803 Is there a specific PR that targets this best?

@possumbilities possumbilities added 🏁 status: ready for work Ready for work and removed 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work labels Aug 20, 2024
@Murdock9803
Copy link
Member

@possumbilities The PR #304 targets the UI improvement for Resource Page. The button alignment and everything is taken care of there 🙌
refactored the html and css for resource page

@possumbilities possumbilities linked a pull request Aug 20, 2024 that will close this issue
7 tasks
@possumbilities possumbilities self-assigned this Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents 🏁 status: ready for work Ready for work
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants