Comments (7)
Please dont think you are making unnecessary PRs. Having some help is very welcome, and Im sure that there a plenty of folks that know a lot more than I do.
I have reservations on adding things too quickly because once you add it, you cannot remove it without a breaking change. So if a solution can be found with the things available, we should use that. If it's an issue that is solved with better documentation, we should do that. And if it's something that needs to go into the lib, then we can do that.
The other thing worth mentioning is that Im slow to adopt accessibility changes because it's very hard to be 100% sure what the best approach is. So it's another area that is slower to be adopted into the project. As I've learned, not implementing a11y is better than implementing a11y poorly.
In closing, I am very grateful for any contributions, and if I seem slow to respond, it's not because I don't want help or think its a bad idea or whatever. I hope it doesn't come across that way.
from vuetensils.
Nice. I'm thinking the best way to provide this is for the role to default to alert
, and inherit the aria-role attribute if provided. That seems a pretty straight forward and intuitive approach. What do you think?
from vuetensils.
Well, say I have a use case where one piece of content is time sensitive and another is not. They both still are "alerting" folks to something, but accessibility wise, the semantics are presented differently. We cannot predict how a developer would like to implement that boolean type functionality, but we can be opinionated about the accessibility semantics.
from vuetensils.
So what would be your theoretical example? <VModal complementary>
?
from vuetensils.
I was thinking <VAlert complementary />
but after meditating on it, I realized I could just use an <aside />
element as it comes with an implied role="complementary"
from my understanding. So I think I can figure something out without this as I can see why it would get hairy.
Also, just want to say thank you for allowing me to work through some of this. I am still learning all the intentions of the library and I am enjoying working with it thus far. So if I make some more potentially unnecessary PR's, please forgive me. Closing this one.
from vuetensils.
Great to know, and thank you for the clarification. It certainly helps.
Just for the record, you have been a great collaborator. I think I am more worried about myself coming off as overly assertive.
I am working in a bit of a silo at my organization these days and don't get to work with many developers in this fashion (i.e Github, and the sort) even though I use Github myself. So I am still new and learning here how to be a productive collaborator on an OS project.
I am definitely going to be on the lookout for some patterns I can contribute to the Cookbook.
from vuetensils.
Thanks. I'm just figuring this stuff out as I go along as well.
Additions to the cookbook would be incredibly valuable. I think that is a big missing value-add for this project for the folks that want a bit more of a fully-implemented solution you would get with a different library like Vuetify.
from vuetensils.
Related Issues (20)
- Intersect question: how to observe many child elements in v-for loop HOT 3
- Feature request? tree component? HOT 2
- Error "export 'version' was not found in 'vue' HOT 6
- Feature: Focusout HOT 2
- Import errors in Vue2 with Vite HOT 7
- [Vue warn]: Error in render: "TypeError: errors.get is not a function" HOT 5
- [Feature Request] allow localization of the buttons of the datepicker HOT 5
- [Feature Request] i18n HOT 1
- [Feature Request] VDate: allow inline usage HOT 3
- vue 3 support ? HOT 2
- Website docs don't show any examples anymore HOT 5
- VAlert dismissible doesn't work with scoped css HOT 5
- Vuetensils: TypeError: Object(...) is not a function HOT 5
- Drawer sliding animation HOT 4
- Browser compatibility HOT 7
- [Feature Request] Support Vue 2.7 by upgrading vue-demi to ^0.13 HOT 6
- Failed to resolve entry for package "vuetensils" HOT 3
- Input still has error styling after changing error conditions HOT 9
- Enable class customisation for individual notifications HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from vuetensils.