Fixed an existing issue with the CEC codebase. Specifically, if using a Nextion display, if EXTEND_KEY_GROUP1 was not set, then the checkButton() function that is compiled did not support the on-screen menu display for legacy uBITX options. The menus would still get activated, but would not display on the Nextion display. Added the #ifdef/#else for USE_SW_SERIAL to the "NOT" EXTEND_KEY_GROUP1 version of checkButton().

Note that there's nothing about this function that requires the keyer implementation.

ubitx_20/ubitx.h

@@ -61,6 +61,7 @@ extern byte I2C_LCD_SECOND_ADDRESS;     //only using Dual LCD Mode
 //==============================================================================
 // User Select feather list
 //==============================================================================
+/*
 //Enable all features
 #define FN_BAND         1 //592
 #define FN_VFO_TOGGLE   1 //78
@@ -84,7 +85,7 @@ extern byte I2C_LCD_SECOND_ADDRESS;     //only using Dual LCD Mode
 #define FN_KEYTYPE      1 //168
 #define FN_ADCMONITOR   1 //516
 #define FN_TXONOFF      1 //58
-
+*/
 /*
 //Test Configuration  (88%)
 #define FN_BAND         0 //592
@@ -110,7 +111,6 @@ extern byte I2C_LCD_SECOND_ADDRESS;     //only using Dual LCD Mode
 #define FN_ADCMONITOR   1 //516
 #define FN_TXONOFF      1 //58
 */
-
 /*
 //Recommended Character LCD Developer  87%
 #define FN_BAND         1 //592
@@ -137,7 +137,6 @@ extern byte I2C_LCD_SECOND_ADDRESS;     //only using Dual LCD Mode
 #define FN_TXONOFF      1 //58
 */
 
-/*
 //Recommended for Nextion, TJC LCD 88%
 #define FN_BAND         1 //600
 #define FN_VFO_TOGGLE   1 //90
@@ -161,7 +160,7 @@ extern byte I2C_LCD_SECOND_ADDRESS;     //only using Dual LCD Mode
 #define FN_KEYTYPE      0 //294
 #define FN_ADCMONITOR   0 //526 //not available with Nextion or Serial UI
 #define FN_TXONOFF      1 //70
-*/
+
 //==============================================================================
 // End of User Select Mode and Compil options
 //==============================================================================
@@ -223,8 +222,9 @@ extern byte I2C_LCD_SECOND_ADDRESS;     //only using Dual LCD Mode
 #define ANALOG_SMETER (A7)  //by KD8CEC
 
 #ifdef USE_ALTKEYER
-  #define DIGITAL_DOT     (11)
-  #define DIGITAL_DASH    (12)
+  #define DIGITAL_DOT     (12)
+  #define DIGITAL_DASH    (11)
+  #define DIGITAL_KEY     (A3)
 #endif
 
 /** 

ubitx_20/ubitx_20.ino

@@ -803,9 +803,17 @@ void checkButton(){
   delay(50);
   if (!btnDown()) //debounce
     return;
- 
+    //for touch screen
+#ifdef USE_SW_SERIAL
+    SetSWActivePage(1);
     doMenu();
 
+    if (isCWAutoMode == 0)
+          SetSWActivePage(0);
+#else
+  doMenu();
+#endif
+  
   //wait for the button to go up again
   while(btnDown()) {
     delay(10);
@@ -1295,6 +1303,7 @@ void initPorts(){
 #ifdef USE_ALTKEYER
   pinMode(DIGITAL_DOT, INPUT_PULLUP);
   pinMode(DIGITAL_DASH, INPUT_PULLUP);
+  if (DIGITAL_KEY != PTT) pinMode(DIGITAL_KEY, INPUT_PULLUP);
 #endif
 
 #ifdef USE_CUSTOM_LPF_FILTER 
@@ -1466,14 +1475,14 @@ void checkAutoSaveFreqMode()
 void loop(){ 
   if (isCWAutoMode == 0){  //when CW AutoKey Mode, disable this process
 #ifdef USE_ALTKEYER
-    // when using the alternate keyer, don't check the PTT if we're in CW mode, because 
-    // the PTT is also a straight key
+    // when using the alternate keyer, don't check the PTT if we're in CW mode--assuming the pin is the same as PTT
+    if (DIGITAL_KEY == PTT) {
       if (!txCAT && (cwMode == 0))
         checkPTT();
-#else
+    } else
+#endif
     if (!txCAT)
       checkPTT();
-#endif
     checkButton();
   }
  else

ubitx_20/ubitx_keyer.ino

@@ -40,19 +40,18 @@ char lastPaddle = 0;
 //reads the analog keyer pin and reports the paddle
 byte getPaddle() {
 #ifdef USE_ALTKEYER
-  // Alternate keyer... but it looks like getPaddle() ever actually gets used, so...
-  if (cwMode > 0) {
-    // PTT functions as straight key, but only if we're in CW mode
-    if (digitalRead(PTT) == LOW) {
+  // Alternate keyer
+  if ((DIGITAL_KEY != PTT) || ((DIGITAL_KEY == PTT) && (cwMode > 0))) {
+    if (digitalRead(DIGITAL_KEY) == LOW) {
       return PADDLE_STRAIGHT;
     }
-  } else {
+  }
   // paddles work whether or not CW mode is actually selected
   int rv = 0;
-    rv |= digitalRead(DIGITAL_DOT) == LOW ? 1 : 0;
-    rv |= digitalRead(DIGITAL_DASH) == LOW ? 2 : 0;
+  rv |= digitalRead(DIGITAL_DOT) == LOW ? PADDLE_DOT : 0;
+  rv |= digitalRead(DIGITAL_DASH) == LOW ? PADDLE_DASH : 0;
   return rv;
-  }  
+
 #else
   int paddle = analogRead(ANALOG_KEYER);
 
@@ -117,12 +116,14 @@ char update_PaddleLatch(byte isUpdateKeyState) {
 #ifdef USE_ALTKEYER
   // One big note... I have no idea how debounce does or doesn't affect this... but there is no debounce right now.  It's conceivable
   // that the normal uBITX use of an analog input for this masks bouncing...
-  if (digitalRead(DIGITAL_DOT) == LOW) tmpKeyerControl |= DIT_L;
-  if (Iambic_Key) {
   if ((digitalRead(DIGITAL_DASH) == LOW)) tmpKeyerControl |= DAH_L;
+  if (Iambic_Key) {
+    if (digitalRead(DIGITAL_DOT) == LOW) tmpKeyerControl |= DIT_L;
   } else {
     // I really would like to have the PTT (during CW mode) just result in key down regardless of Iambic, but this doesn't do that yet...
-    if ((cwMode > 0) && (digitalRead(PTT) == LOW)) tmpKeyerControl |= DIT_L;
+    if ((DIGITAL_KEY != PTT) || ((DIGITAL_KEY == PTT) && (cwMode > 0))) {
+      if (digitalRead(DIGITAL_KEY) == LOW) tmpKeyerControl |= DAH_L;
+    }  
   }
 #else
   int paddle = analogRead(ANALOG_KEYER);
@@ -252,10 +253,9 @@ void cwKeyer(void){
   
       Check_Cat(2);
     } //end of while
-  }
-  else{
+  } else {
     while(1){
-      if (update_PaddleLatch(0) == DIT_L) {
+      if (update_PaddleLatch(0) == DAH_L) {
         // if we are here, it is only because the key is pressed
         if (!inTx){
           //DelayTime Option
@@ -267,7 +267,7 @@ void cwKeyer(void){
         }
         cwKeydown();
         
-        while ( update_PaddleLatch(0) == DIT_L ) 
+        while ( update_PaddleLatch(0) == DAH_L ) 
           delay_background(1, 3);
           
         cwKeyUp();