Skip to content
This repository was archived by the owner on Jul 26, 2022. It is now read-only.

added Feature: :disable Property #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

stevenbuehner
Copy link

I was missing the feature to disable the input-field itself. Most vue bootstrap components that are using input-fields contain a :disabled property.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 48

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 69.388%

Totals Coverage Status
Change from base Build 38: 0.0%
Covered Lines: 47
Relevant Lines: 68

💛 - Coveralls

@jjohnson1994
Copy link

Any chance of this being merged? Need this feature

jhalag added a commit to jhalag/vue-bootstrap-typeahead that referenced this pull request Apr 16, 2019
jhalag added a commit to jhalag/vue-bootstrap-typeahead that referenced this pull request Apr 16, 2019
@entraigas
Copy link

I need this feature too. Please can you merge it?

@qadirfarooq
Copy link

is this feature is available now ? ?

@mattzollinhofer
Copy link

Hey, I'm looking to revive this project, but I haven't been able to track down the author. So, I've forked it over to here: https://github.com/mattzollinhofer/vue-typeahead-bootstrap.

I'll try to take a look at this PR soon and see if I can get it merged as it seems like there's a number of folks who need this to be fixed and it follows spirit of integrating with bootstrap.

@xerardoo
Copy link

xerardoo commented Mar 13, 2020

An option is access to the input via $el, example:

<vue-bootstrap-typeahead
           inputClass="input-property"
           ...
/>
 mounted() {
            this.$el.querySelector(".input-property").setAttribute('disabled', true)`
}

@mattzollinhofer
Copy link

@stevenbuehner: thanks again for this PR -- I just merged this into the new repo: mattzollinhofer/vue-typeahead-bootstrap@660f49a, I hope it's still helpful!

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

Successfully merging this pull request may close these issues.

7 participants