This repository has been archived by the owner. It is now read-only.

Can't use valueattr and label #22

Closed
fballiano opened this Issue Oct 18, 2014 · 4 comments

Comments

Projects
None yet
2 participants
@fballiano

fballiano commented Oct 18, 2014

Hi,
if I've a component like this:

<paper-dropdown-menu label='Category' selected='1' valueattr='value' >
 <paper-item value="1">label1</paper-item>
 <paper-item value="2">label2</paper-item>
 <paper-item value="3">label3</paper-item>
</paper-dropdown-menu>

when i click on it and select "label1" the dropdown will close but it will show the value, not the label.

@morethanreal

This comment has been minimized.

Show comment
Hide comment
@morethanreal

morethanreal Oct 21, 2014

This is by design. When an item is selected, it will show the selected value instead of the label, like the select element.

morethanreal commented Oct 21, 2014

This is by design. When an item is selected, it will show the selected value instead of the label, like the select element.

@fballiano

This comment has been minimized.

Show comment
Hide comment
@fballiano

fballiano Oct 21, 2014

what?
the html "select" doesn't show the value, it shows the selected option's label:
http://jsbin.com/raxobusuqu/1/edit?html,output

fballiano commented Oct 21, 2014

what?
the html "select" doesn't show the value, it shows the selected option's label:
http://jsbin.com/raxobusuqu/1/edit?html,output

@morethanreal

This comment has been minimized.

Show comment
Hide comment
@morethanreal

morethanreal Oct 21, 2014

I see what you mean, I thought you wanted the label attribute to show even when an item is selected. More specific description of the expected output would be helpful next time :)

morethanreal commented Oct 21, 2014

I see what you mean, I thought you wanted the label attribute to show even when an item is selected. More specific description of the expected output would be helpful next time :)

@morethanreal morethanreal reopened this Oct 21, 2014

@morethanreal

This comment has been minimized.

Show comment
Hide comment
@morethanreal

morethanreal Oct 30, 2014

Should be fixed.

morethanreal commented Oct 30, 2014

Should be fixed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.