Closed suzidao closed 10 years ago
Thanks, Susanne. On first glance, I notice you're redefining the value of the @padding-small-vertical variable. That's problematic as that variable is used by other components (buttons, input-groups, etc.). Please create a custom variable for use in pagination. Thanks!
Hi Robb, I've rolled back the @padding-small-vertical variable and added @pagination-padding-small-vertical. Please let me know if there are any more discrepancies. This is my first time using GitHub and I'm still trying to get the hang of the workflow. You may notice a few Modal Window-related edits in this branch; I began work on that before I remembered that you wanted individual branches for each component. I'll have that one done later tonight.
Thanks, Susanne. I didn't notice the modal changes, but can you remove anything related to modals so it can go in its own pull request? Thanks.
Hi Robb, all modal changes have been removed. I'll submit the modal pull request shortly.
Hi, Susanne.
Sorry for the delay in getting this reviewed. I've been busy trying to get the PatternFly site up!
Can you please make the following changes:
Sorry for not providing clear direction on the coding standards! I will document them in the readme.
Thanks, Robb.
Hi Robb,
I've updated the pager and pagination files to comply with coding standards. Please let me know if further changes are needed.
Thanks, Susanne
Hi Robb,
Here are the pager and pagination components for RCUE.
Commit Comments: Added pagination test page Pager style updates Pagination refinement Small size and various states Pagination hover bug fix attempt Pagination hover bug fix attempt: removed extraneous .button-variant mixins Pagination hover bug fix - final Pager disabled active bug fix Pagination - added small pager styles Pagination - added headers to sample page Cross-browser edits