Skip to content

Conversation

@dehru
Copy link
Contributor

@dehru dehru commented Dec 15, 2020

What does this PR do?

This PR reduces the chattiness within the soql builder ui, causing some unwanted input field strangeness, where characters are suddenly removed as i type.

What issues does this PR fix or reference?

@W-8555599@

Functionality Before

Notice as i'm typing into the input, adding or removing characters, multiple characters keep getting added/removed suddenly.

flickr-before

Functionality After

Notice as I'm typing into the input, adding or removing characters is smooth.

flickr-after

@dehru dehru changed the title reducing the need to re-render the ui if soql statement has not changed Reducing the message chattiness between the webview and vscode custom editor Dec 15, 2020
@dehru dehru changed the title Reducing the message chattiness between the webview and vscode custom editor Preventing webview re-rendering when soql statement has not changed Dec 15, 2020
Base automatically changed from jhork/where-ui to feature/where December 16, 2020 17:58
@dehru dehru force-pushed the dehru/bug/where-ui-flickr branch from 2770958 to d6e350f Compare January 4, 2021 20:37
@dehru dehru merged commit 6ac002b into feature/where Jan 4, 2021
@dehru dehru deleted the dehru/bug/where-ui-flickr branch January 4, 2021 20:39
dehru added a commit that referenced this pull request Feb 2, 2021
* Jhork: Support for Simple WHERE Expressions (#101)
* move rule to css common (#108)
* Preventing webview re-rendering when soql statement has not changed (#105)
* Revert "Disable WHERE in model (#113)"
* Type-aware where (#109)
* Recognize recoverable where errors (#116)
* Operator and Criteria Error Tooltips (#122)
* Support IN, NOT IN, INCLUDES, EXCLUDE operators (#123)
jgellin-sf pushed a commit that referenced this pull request Feb 3, 2021
…105)

* reducing the need to re-render the ui if soql statement has not changed
* move check to reduce message parsing
dehru added a commit that referenced this pull request Feb 5, 2021
* Jhork: Support for Simple WHERE Expressions (#101)
* move rule to css common (#108)
* Preventing webview re-rendering when soql statement has not changed (#105)
* Revert "Disable WHERE in model (#113)"
* Type-aware where (#109)
* Fix preview not updating when autosave is off - issue 120 (#125)
* Recognize recoverable where errors (#116)
* Operator and Criteria Error Tooltips (#122)
* Support IN, NOT IN, INCLUDES, EXCLUDE operators (#123)
* Add smarter completin on GROUP BY clause and WHERE literals (#126)
* Feature/custom select (#148)
* Jhork/custom select fields (#111)
* Jhork/custom select from (#130)
* Jhork/custom select order by (#134)
* set the error overlay to be on top of all other form elements (#150)
* prepare to publish 0 .0.30 (#151)
* Feature/where (#154)
* Jhork: Support for Simple WHERE Expressions (#101)
* move rule to css common (#108)
* Preventing webview re-rendering when soql statement has not changed (#105)
* Revert "Disable WHERE in model (#113)"
* Type-aware where (#109)
* Recognize recoverable where errors (#116)
* Operator and Criteria Error Tooltips (#122)
* Support IN, NOT IN, INCLUDES, EXCLUDE operators (#123)
* Revert "Feature/where (#154)" (#157)
* Add support for line comments ( `//` ) at the top of SOQL files (#132)
* Jhork/where custom select (#161)
* Fix: 'undefined' in SOQL string (#162)
* Clear tooltip when object changes, fix empty soql statement error blocker (#165)
* Track to force render on condition change - fix rebase conflict (#166)
* do not send event when user still adding input' (#169)
* Jhork/clear modifier field bug fix (#168)
* handle a option selection event (#178)
* cler tooltip error when X is clicked (#180)
* Jhork/workflow updates (#163)
* Bump versions for release (#181)

Co-authored-by: Jonny Hork <jhork@salesforce.com>
Co-authored-by: dehru <599418+dehru@users.noreply.github.com>
Co-authored-by: Dehru Cromer <dehru.cromer@salesforce.com>
Co-authored-by: Fernando Dobladez <fernandodobladez@salesforce.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant