public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "gaoliming" <gaoliming@byosoft.com.cn>
To: <devel@edk2.groups.io>, <gechao@greatwall.com.cn>,
	<dandan.bi@intel.com>, <eric.dong@intel.com>
Cc: <jian.j.wang@intel.com>, <hao.a.wu@intel.com>
Subject: 回复: [edk2-devel] [PATCH] MdeModulePkg/DisplayEngineDxe: Fix Help information shadow problem
Date: Tue, 29 Dec 2020 09:45:18 +0800	[thread overview]
Message-ID: <002601d6dd84$43854f80$ca8fee80$@byosoft.com.cn> (raw)
In-Reply-To: <d7f8114b79f05db866180dc8017b291856d6f175.1609143528.git.gechao@greatwall.com.cn>

Chao:
  Please submit BZ in https://bugzilla.tianocore.org/, and fill more detail
scription. It will be helpful for others to understand this problem.  

Thanks
Liming
> -----邮件原件-----
> 发件人: bounce+27952+69488+4905953+8761045@groups.io
> <bounce+27952+69488+4905953+8761045@groups.io> 代表
> gechao@greatwall.com.cn
> 发送时间: 2020年12月28日 17:12
> 收件人: dandan.bi@intel.com; eric.dong@intel.com
> 抄送: devel@edk2.groups.io; jian.j.wang@intel.com; hao.a.wu@intel.com;
> gechao <gechao@greatwall.com.cn>
> 主题: [edk2-devel] [PATCH] MdeModulePkg/DisplayEngineDxe: Fix Help
> information shadow problem
> 
> From: gechao <gechao@greatwall.com.cn>
> 
> Fix residual display problem in help information section with different
> length string. When switching between different items, the help inform-
> ation part may remain the last displayed help information content.
> Example:
>     Last: ABCDEFGH, New:  1234567
>     Result: 1234567H
> 
> Signed-off-by: gechao <gechao@greatwall.com.cn>
> ---
>  .../Universal/DisplayEngineDxe/FormDisplay.c  | 27
> +++++++++++++++++++
>  1 file changed, 27 insertions(+)
> 
> diff --git a/MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c
> b/MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c
> index a3898dc2ec..abb4843a73 100644
> --- a/MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c
> +++ b/MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c
> @@ -3061,6 +3061,15 @@ UiDisplayMenu (
>        //
> 
>        if (!MultiHelpPage) {
> 
>          for (Index = 0; Index < HelpLine; Index++) {
> 
> +          //
> 
> +          // Clear whole line first, Clear the last displayed
information.
> 
> +          //
> 
> +          PrintStringAtWithWidth (
> 
> +            gStatementDimensions.RightColumn - gHelpBlockWidth,
> 
> +            Index + TopRow,
> 
> +            gEmptyString,
> 
> +            gHelpBlockWidth
> 
> +            );
> 
>            PrintStringAtWithWidth (
> 
>              gStatementDimensions.RightColumn - gHelpBlockWidth,
> 
>              Index + TopRow,
> 
> @@ -3080,6 +3089,15 @@ UiDisplayMenu (
>        } else  {
> 
>          if (HelpPageIndex == 0) {
> 
>            for (Index = 0; Index < RowCount - HelpBottomLine; Index++) {
> 
> +            //
> 
> +            // Clear whole line first, Clear the last displayed
information.
> 
> +            //
> 
> +            PrintStringAtWithWidth (
> 
> +              gStatementDimensions.RightColumn - gHelpBlockWidth,
> 
> +              Index + TopRow,
> 
> +              gEmptyString,
> 
> +              gHelpBlockWidth
> 
> +              );
> 
>              PrintStringAtWithWidth (
> 
>                gStatementDimensions.RightColumn - gHelpBlockWidth,
> 
>                Index + TopRow,
> 
> @@ -3090,6 +3108,15 @@ UiDisplayMenu (
>          } else {
> 
>            for (Index = 0; (Index < RowCount - HelpBottomLine -
> HelpHeaderLine) &&
> 
>                (Index + HelpPageIndex * (RowCount - 2) + 1 < HelpLine);
> Index++) {
> 
> +            //
> 
> +            // Clear whole line first, Clear the last displayed
information.
> 
> +            //
> 
> +            PrintStringAtWithWidth (
> 
> +              gStatementDimensions.RightColumn - gHelpBlockWidth,
> 
> +              Index + TopRow + HelpHeaderLine,
> 
> +              gEmptyString,
> 
> +              gHelpBlockWidth
> 
> +              );
> 
>              PrintStringAtWithWidth (
> 
>                gStatementDimensions.RightColumn - gHelpBlockWidth,
> 
>                Index + TopRow + HelpHeaderLine,
> 
> --
> 2.25.1
> 
> 
> 
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#69488): https://edk2.groups.io/g/devel/message/69488
> Mute This Topic: https://groups.io/mt/79282846/4905953
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub
> [gaoliming@byosoft.com.cn]
> -=-=-=-=-=-=
> 




      reply	other threads:[~2020-12-29  1:45 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-28  9:11 [PATCH] MdeModulePkg/DisplayEngineDxe: Fix Help information shadow problem gechao
2020-12-29  1:45 ` gaoliming [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='002601d6dd84$43854f80$ca8fee80$@byosoft.com.cn' \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox