After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 764445 - authPrompt: PAM message wrap_line failed
authPrompt: PAM message wrap_line failed
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: login-screen
3.10.x
Other Linux
: Normal critical
: ---
Assigned To: Ray Strode [halfline]
gnome-shell-maint
: 745082 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2016-04-01 08:25 UTC by tyang
Modified: 2016-10-14 13:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
wrap_line failed (7.61 KB, text/html)
2016-04-01 08:30 UTC, tyang
  Details
make line-wrap work (1.66 KB, patch)
2016-04-02 04:09 UTC, tyang
none Details | Review
patch for fixing wrap_line issue (1.26 KB, patch)
2016-04-06 11:52 UTC, tyang
none Details | Review
Patch based on latest git code (1.60 KB, patch)
2016-10-13 04:49 UTC, tyang
committed Details | Review

Description tyang 2016-04-01 08:25:12 UTC
Issue is same with bgo#705037:

We got warning message from pam which should be shown as multiple lines (then length of message is longer than entry of password entry of course). But we can see this message is shown just in one line and cut, ends up with "..."

I'm sure we got the bgo#705037 involved(gnome 3.10)

So I add following code into gnome-shell/js/gdm/authPrompt.js:
this._message.clutter_text.ellipsize = Pango.EllipsizeMode.NONE;

to make sure that we don't take ellipsize into consideration so that we can make line_wrap works.

Although message are shown in two lines, I got wired characters in the second line.

So I am wondering:
Did we really solve the issue in bgo#705037?

Thanks
Comment 1 tyang 2016-04-01 08:30:09 UTC
Created attachment 325131 [details]
wrap_line failed
Comment 2 tyang 2016-04-01 08:34:39 UTC
(In reply to tyang from comment #1)
> Created attachment 325131 [details]
> wrap_line failed

Please ignore this attachment,sorry
Comment 3 tyang 2016-04-02 04:09:51 UTC
Created attachment 325203 [details] [review]
make line-wrap work

To make wrap-line work by setting clutter_text.line_wrap to true,
we need to ensure that st-label has installed the property line-wrap
already.Then we also need to set ellipsize to default.
Comment 4 tyang 2016-04-02 06:02:44 UTC
committed it on github:
https://github.com/GNOME/gnome-shell/pull/7
Comment 5 tyang 2016-04-06 11:52:35 UTC
Created attachment 325476 [details] [review]
patch for fixing wrap_line issue

The point is we need to confirm that ellipsize is set to false ,so that we can truly make wrap_line happen, otherwise the ellipsize will have a higher priority than the wrap_line property in clutter_text.

I wish this could be handled soon, so if you think this is correct, please let me know where I can push this patch to on github...

PS1: This patch is based on gnome 3.10, so the changes in theme file should be omitted.

PS2: You can reproduce the issue by changing the password when next time you try to log into the gnome, and set a password which is against the complexity of pam/AD.

Thanks
Comment 6 Ray Strode [halfline] 2016-10-12 13:49:29 UTC
*** Bug 745082 has been marked as a duplicate of this bug. ***
Comment 7 tyang 2016-10-13 04:49:20 UTC
Created attachment 337554 [details] [review]
Patch based on latest git code

This patch is created with git://git.gnome.org/gnome-shell
and with gnome-style commit message from https://wiki.gnome.org/Newcomers/CodeContributionWorkflow#Commit_guidelines

Thanks
Comment 8 Ray Strode [halfline] 2016-10-13 14:24:27 UTC
Review of attachment 337554 [details] [review]:

thanks.  do you have commit access?
Comment 9 tyang 2016-10-14 05:31:17 UTC
(In reply to Ray Strode [halfline] from comment #8)
> Review of attachment 337554 [details] [review] [review]:
> 
> thanks.  do you have commit access?

No, I don't
how can I get the access to that?

Thanks
Comment 10 Ray Strode [halfline] 2016-10-14 13:27:45 UTC
well for now I'll just push it myself. If you keep doing patches though, we can get you access to make pushing easier.
Comment 11 Ray Strode [halfline] 2016-10-14 13:31:08 UTC
Thanks much !