-
-
Notifications
You must be signed in to change notification settings - Fork 137
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
Comments
Good catch. Apparently, the misalignment of download button is only in open data handbook resource and no other. 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 Here's the repo url that was compromised. If this was intentional, then I would suggest to just remove the 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. |
@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. |
Updated issue title and description to address the main point. |
@Lakshay-Pruthi Clicking on the button previously led to a website that is no longer available on the internet, namely lawrent.com. |
@Lakshay-Pruthi, you only need to organize the CSS styles, and your exploration in the website is really appreciative. |
Hey @Lakshay-Pruthi and @Keerthivardhan1, |
Please assign this issue to me. |
@abhishekshah5486 The UI for the page has major improvements done, and the issue has become irrelevant since then. |
@Murdock9803 Is there a specific PR that targets this best? |
@possumbilities The PR #304 targets the UI improvement for Resource Page. The button alignment and everything is taken care of there 🙌 |
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.

Unnecessary code which can be deleted

Instead of this we can use flex properties in style.css
Reproduction
Expectation
It should be properly aligned in the centre.
Resolution
The text was updated successfully, but these errors were encountered: