Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id C0D5F200D2D for ; Fri, 27 Oct 2017 18:13:05 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id BF785160BDC; Fri, 27 Oct 2017 16:13:05 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 109BA1609E9 for ; Fri, 27 Oct 2017 18:13:04 +0200 (CEST) Received: (qmail 37128 invoked by uid 500); 27 Oct 2017 16:13:04 -0000 Mailing-List: contact dev-help@directory.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Apache Directory Developers List" Delivered-To: mailing list dev@directory.apache.org Received: (qmail 37117 invoked by uid 99); 27 Oct 2017 16:13:04 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 27 Oct 2017 16:13:04 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id 15262C0A12 for ; Fri, 27 Oct 2017 16:13:03 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -99.202 X-Spam-Level: X-Spam-Status: No, score=-99.202 tagged_above=-999 required=6.31 tests=[KAM_ASCII_DIVIDERS=0.8, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, USER_IN_WHITELIST=-100] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id a5XVf-JyfYzz for ; Fri, 27 Oct 2017 16:13:02 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTP id 748C960EF1 for ; Fri, 27 Oct 2017 16:13:01 +0000 (UTC) Received: from jira-lw-us.apache.org (unknown [207.244.88.139]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id C829CE0EF0 for ; Fri, 27 Oct 2017 16:13:00 +0000 (UTC) Received: from jira-lw-us.apache.org (localhost [127.0.0.1]) by jira-lw-us.apache.org (ASF Mail Server at jira-lw-us.apache.org) with ESMTP id 2594E212FA for ; Fri, 27 Oct 2017 16:13:00 +0000 (UTC) Date: Fri, 27 Oct 2017 16:13:00 +0000 (UTC) From: "Lothar Haeger (JIRA)" To: dev@directory.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Comment Edited] (DIRSTUDIO-1156) Text Editor usability improvements MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 archived-at: Fri, 27 Oct 2017 16:13:05 -0000 [ https://issues.apache.org/jira/browse/DIRSTUDIO-1156?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16222614#comment-16222614 ] Lothar Haeger edited comment on DIRSTUDIO-1156 at 10/27/17 4:12 PM: -------------------------------------------------------------------- So I've changed then HexDialog and TextDialog classes as described above, just the "wrap lines" check box has become a toggle button instead. Please see attached DIRSTUDIO-1156-1.patch [~elecharny]: would you be so kind and review the patch and apply to trunk if you see fit? PS: my French did not come up with a good translation to "wrap lines" or "Zeilenumbruch" yet, please feel free to change the last line of the patch (-> message_fr.properties) was (Author: lhaeger): So I've changed then HexDialog and TextDialog classes as described above, just the "wrap lines" check box has become toggle button instead. Please see attached DIRSTUDIO-1156-1.patch [~elecharny]: would you be so kind and review the patch and apply to trunk if you see fit? PS: my French did not come up with a good translation to "wrap lines" or "Zeilenumbruch" yet, please feel free to change the last line of the patch (-> message_fr.properties) > Text Editor usability improvements > ---------------------------------- > > Key: DIRSTUDIO-1156 > URL: https://issues.apache.org/jira/browse/DIRSTUDIO-1156 > Project: Directory Studio > Issue Type: Improvement > Components: studio-ldapbrowser > Affects Versions: 2.0.0-M13 > Environment: Windows > Reporter: Lothar Haeger > Assignee: Lothar Haeger > Priority: Minor > Labels: usability > Attachments: DIRSTUDIO-1156_1.patch > > > As an IDM developer I work a lot with XML blobs stored in binary attributes. Those values open in Hex Editor initially, which has a "Edit as Text..." button to open the value in a Text Editor if possible (i.e. not containing any non-printable characters). There are a few aspects to improve here, especially when working with long lines of text or overall large texts: > * the Text Editor cannot be maximized on all platforms. SWT.RESIZE is set, but is a lot less convenient to use if you need as much screen area as possible. Possible solution: add SWT.MAX when creating the dialog > * the Text Editor does not support line wrapping. XML or JSON data is often stored without line breaks, which requires cumbersome horizontal scrolling. Possible solution: add a "wrap lines" check box (display only, not adding line breaks to the value text, inactive by default) > * the Text Editor dialog has a relatively small default size, a ~50% wider/higher seems reasonable > * the button to open the Text Editor from Hex Editor is always displayed, even if the binary value is not editable (e.g. contains an image). Only when the button has been pressed a check is performed and an error message shown. It would be more intuitive to hide the button if a binary value cannot be edited as text in the first place. -- This message was sent by Atlassian JIRA (v6.4.14#64029)