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

Shortcuts behavior sometimes weird #641

Closed
metas-mk opened this issue Apr 14, 2017 · 6 comments
Closed

Shortcuts behavior sometimes weird #641

metas-mk opened this issue Apr 14, 2017 · 6 comments
Assignees
@damianprzygodzki @metas-lc
Labels
branch:master branch:release type:bug
Milestone
2017-17

Comments

@metas-mk
Copy link
Member

Type of issue

Bug

Current behavior

Sometimes when toggling through the menues, it happens that the shortcut for a menu needs 2 presses to open the menu. It has then close of old menu in 1st step, and open the menu in 2nd.

Expected behavior

Shall always work with 1 press/ step.

Steps to reproduce

Try the following in Sales Order window /window/143/1005041
ctrl+1 OK
ctrl+5 OK
ctrl+3 OK
ctrl+1 OK
ctrl+5 NOK (have to press twice, 1st one closes menu1, 2nd one opens menu5)
@metas-mk metas-mk added this to the 2017-17 milestone Apr 14, 2017
@damianprzygodzki damianprzygodzki self-assigned this Apr 14, 2017
damianprzygodzki added a commit that referenced this issue Apr 14, 2017
@damianprzygodzki
#641 Clearing sidelist tab after closing with shortcut
7ad1f59
@cadavre
Copy link
Contributor

cadavre commented Apr 20, 2017

@metas-lc can be closed?

@metas-lc metas-lc self-assigned this Apr 20, 2017
@metas-lc
Copy link

IT
sales order window, tried ctrl+1,2,3,4,5 and 6 in multiple orders, including 1,5,3,1,5 and they work fine, opening the menus directly, without having to press twice. OK

question:
when i press ctrl+1,2 and 3 twice, the menu opens and closes
but when i press ctrl+4,5 and 6 twice, the menu just refreshes, does not close. Is that OK?

i'll create a new task for my question and close this one since it's ok

@damianprzygodzki
Copy link
Contributor

@metas-lc Please, do not open another issue. It is bug followed by this one. Im gonna do quickfix.

@metas-lc
Copy link

@damianprzygodzki ok, thank you

@metas-lc metas-lc reopened this Apr 20, 2017
damianprzygodzki added a commit that referenced this issue Apr 20, 2017
@damianprzygodzki
#641 Fixing current sideListTab
252bbd5
@damianprzygodzki
Copy link
Contributor

damianprzygodzki commented Apr 20, 2017 •
edited
Loading

@metas-lc Done

@metas-lc
Copy link

IT2
retested the testcase above. Works fine

metas-ts added a commit to metasfresh/metasfresh that referenced this issue Apr 28, 2017
@metas-ts
adding additional items to the RNs
015fe73 
[#1382](#1382) Make Packing Item editable in Manufacturing Order Header
[#323](metasfresh/metasfresh-webui-api-legacy#323) Manufacturing Issue/Receipt: receipt lines shall be displayed first
[#689](metasfresh/metasfresh-webui-frontend-legacy#689) Image widget is missing right border
[#326](metasfresh/metasfresh-webui-api-legacy#326) technical: Remove JSONProcessInstanceResult deprecated properties
[#680](metasfresh/metasfresh-webui-frontend-legacy#680) Navigation breadcrumb vs. Navigation Menu
[#329](metasfresh/metasfresh-webui-api-legacy#329) Issue HU 404 not found#
[#688](metasfresh/metasfresh-webui-frontend-legacy#688) Main page /dashboard is broken
[#667](metasfresh/metasfresh-webui-frontend-legacy#667) Change /rest/api/pattribute request message
[#319](metasfresh/metasfresh-webui-api-legacy#319) Manufacturing Issue/Receipt: show BOM line attributes if any
[#317](metasfresh/metasfresh-webui-api-legacy#317) Manufacturing Issue/Receipt: drop StatusInfo column
[#248](metasfresh/metasfresh-webui-frontend-legacy#248) Handle Password Fields in WebUI
[#678](metasfresh/metasfresh-webui-frontend-legacy#678) Get rid of deprecated static docNo input in header.
[#151](metasfresh/metasfresh-webui-api-legacy#151) grid view websocket notifications: fullyChanged shall not be set when changedIds is set
[#664](metasfresh/metasfresh-webui-frontend-legacy#664) process doesn't open directly
[#311](metasfresh/metasfresh-webui-api-legacy#311) Switching language does not translate document lookup values
[#310](metasfresh/metasfresh-webui-api-legacy#310) Switching language does not translate document references
[#306](metasfresh/metasfresh-webui-api-legacy#306) Address missing in 'Adresse' for bpartner created on-the-fly
[#673](metasfresh/metasfresh-webui-frontend-legacy#673) Wrong report file endpoint is called for processes which are started from main menu
[#1347](#1347) technical: ModelClassGenerator shall generate BigDecimal.ZERO instead of Env.ZERO
[#880](#880) Improve IProcessPrecondition framework
[#641](metasfresh/metasfresh-webui-frontend-legacy#641) Shortcuts behavior sometimes weird
[#644](metasfresh/metasfresh-webui-frontend-legacy#644) Creating a new Discount record error
[#649](metasfresh/metasfresh-webui-frontend-legacy#649) windowId shall be handled as a string and not as a number
[#596](metasfresh/metasfresh-webui-frontend-legacy#596) red line appears on document only after refresh
[#433](metasfresh/metasfresh-webui-frontend-legacy#433) ShipmentSchedule: Only displayed after "No data" showing up
[#270](metasfresh/metasfresh-webui-api-legacy#270) cannot add trading unit with batch entry after using add new
[#537](metasfresh/metasfresh-webui-frontend-legacy#537) KPI: implement a non intrusive way to display KPI related errors
[#268](metasfresh/metasfresh-webui-api-legacy#268) HU attributes shall always provide the WidgetType along with the data/changes
[#579](metasfresh/metasfresh-webui-frontend-legacy#579) DocumentNo field is not updated

me-45
hairy tinkerbell struck again
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@damianprzygodzki damianprzygodzki

@metas-lc metas-lc

Labels
branch:master branch:release type:bug
Projects
None yet
Milestone
2017-17
Development

No branches or pull requests
5 participants
@cadavre @metas-ts @damianprzygodzki @metas-mk @metas-lc