Skip to content
This repository has been archived by the owner on Nov 26, 2021. It is now read-only.
This repository has been archived by the owner on Nov 26, 2021. It is now read-only.

Cursor 'Pointer' CSS Not Applied Consistently #80

Closed
jwbats opened this issue Aug 17, 2021 · 7 comments
Closed

Cursor 'Pointer' CSS Not Applied Consistently #80

jwbats opened this issue Aug 17, 2021 · 7 comments

Comments

@jwbats
Copy link

jwbats commented Aug 17, 2021

  • Vue.js version: 3.0.91
  • consumed using: WebPack
  • FULL error message: N/A
  • relevant code: N/A
  • steps for reproducing the issue: use paginator on page that has Bootstrap 4 on it

I'm using this paginator on a page that's powered by Bootstrap 4.

For the numbered buttons (1, 2, 3, etc.), it will automatically apply a cursor 'pointer' styling, because they have 'a' tags with the role="button" attribute.

For the prev/next buttons (<<, <, >, >>) it will not do that, because 'a' elements do not have the role="button" attribute.

Both numbered and prev/next buttons, are all clickable elements. It would be nice if they were all styled to have cursor 'pointer' by default, rather than only some of them getting that via Bootstrap 4.

@jwbats
Copy link
Author

jwbats commented Aug 17, 2021

I've temporarily solved it by adding this to my parent component:

/deep/ .VuePagination a.page-link {
    cursor:pointer;
}

@matfish2
Copy link
Owner

a tags were replaced with button elements, so this is resolved

@jwbats
Copy link
Author

jwbats commented Oct 29, 2021

When will the new version be available on npmjs?

@matfish2
Copy link
Owner

It is already available https://www.npmjs.com/package/v-pagination-3

@jwbats
Copy link
Author

jwbats commented Oct 29, 2021

That's another package and it's for Vue 3.

I'm using Vue 2 and have created this issue under the vue-pagination-2 repo, where the vue-pagination-2 npm page (currently still 3.0.91) points to.

@matfish2
Copy link
Owner

matfish2 commented Oct 29, 2021

You wrote "Vue.js version: 3.0.91", I suppose you meant vue-pagination version. I will have to change on the vue2 (master) branch. Reopening for now

@matfish2 matfish2 reopened this Oct 29, 2021
@jwbats
Copy link
Author

jwbats commented Oct 29, 2021

Ah, sorry... my bad. Indeed I did mean that.

Although, in my defense... it's because that's the default template that 'new issue' will show you in this repo. That should probably explicitly read 'vue2 version'. And it should probably also request the 'vue-pagination-2' version.

That would clear things up.

Anyway, tnx for fixing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants