Skip to content
This repository has been archived by the owner on Mar 14, 2020. It is now read-only.

[Property View] Fix bug: Add/Remove SelectMenu option refresh issue. #275

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

[Property View] Fix bug: Add/Remove SelectMenu option refresh issue. #275

wants to merge 1 commit into from

Conversation

xuqingkuang
Copy link
Contributor

No description provided.

@xuqingkuang
Copy link
Contributor Author

@zhizhangchen The code updated.

  1. Fixed add/remove refresh in layoutView and codeView issue.
  2. Make ID of select menu table flexible, previous implementation is fixed to 'selectOption'.

Problems:

  1. Focus lost after change text/value.
  2. Current implementation is redraw whole table to refresh, the performance a bit slower than operate element directly.

Please see the code for more information.

@grgustaf
Copy link
Contributor

Fixed up the minor issues and merged to stable branch for the moment at least. I'll decide later today whether i'm merging that into master (reverting event handlers) or what.

@grgustaf
Copy link
Contributor

grgustaf commented Sep 1, 2012

Merged into master.

@grgustaf grgustaf closed this Sep 1, 2012
@grgustaf grgustaf reopened this Sep 1, 2012
@grgustaf
Copy link
Contributor

grgustaf commented Sep 1, 2012

I'll reopen just for Shane to see a little history. Also, please have Shane look at the comment history from the previous lists PR so he can see the kind of thought and bug fixing that went in before we finally merged it.

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.

2 participants