Skip to content

Conversation

@satishkn
Copy link
Contributor

@satishkn satishkn commented Mar 25, 2013

  • Fix for the following bug:

[OPEN-342] Bug with single-click - reproducible in 1.5.1
as posted in the forum at:
http://ext.ensible.com/forum/viewtopic.php?f=3&t=634

Implementation:
Handling 'keyup' event (just ESC key), so event can be created with the next click after ESC is pressed by the user when context menu is shown .

Tested the following scenarios (on windows XP):

  1. FF & Chrome -> Create event in basic.html sample after ESC key and with single click.

Noticed the following:

  1. In chrome, 'keypress' and 'keydown' events aren't triggered for ESC key, whereas in FF it is triggering.

This change is Reviewable

[OPEN-342] Bug with single-click - reproducible in 1.5.1
as posted in the forum at:
http://ext.ensible.com/forum/viewtopic.php?f=3&t=634
[OPEN-342] Bug with single-click - reproducible in 1.5.1
as posted in the forum at:
http://ext.ensible.com/forum/viewtopic.php?f=3&t=634

Implementation:
Handling 'keyup' event (just ESC key) so event can be created with the next click after ESC is pressed by the user.

Tested the following scenarios (on windows XP):
1)  FF & Chrome -> Create event in basic.html sample after ESC key and with single click.

Noticed the following:
1) In chrome, 'keypress' and 'keydown' events aren't triggered for ESC key, whereas in FF it is triggering.
Conflicts:
	src/calendar/view/AbstractCalendar.js
@@ -22,7 +23,8 @@ Ext.define('Extensible.calendar.view.AbstractCalendar', {
'Extensible.calendar.dd.DropZone',
'Extensible.form.recurrence.RangeEditWindow'
],

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please discard whitespace-only changes prior to committing

@bmoeskau
Copy link
Owner

Once you clean up these changes I'll merge the PR.

…rovide a handler on: “eventdblclick” event
[OPEN-334] Month view: "+X more..." link missing. Details about the bug can be reviewed at:
http://ext.ensible.com/forum/viewtopic.php?f=3&t=613&sid=9f1e63e3221d5e8bb110e99d2f4f3f5e

Implementation is to recalculate the no of events to display when the month is changed.

Steps to verify:
in the current TestApp example, load the current month view for march 2013 it has all the test events.  Click the < button to go to feb, then resize to make the browser smaller, then click > to go back to march, the + more links should be displayed properly (not hidden).
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

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.

2 participants