Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for the bug: [OPEN-342] Bug with single-click - reproducible in 1.5.1 #57

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

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).
@sonarcloud
Copy link

sonarcloud bot commented Sep 15, 2021

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.

None yet

2 participants