Bookmarks: Note Field from Bookmarks Are Not Aligned

[paulcalabro]

Describe the bug
When viewing the bookmarks in list view, the note fields are not aligned.

To Reproduce
Steps to reproduce the behavior:

1. Create multiple bookmarks
2. Click on list view in the UI
3. See error

Expected behavior
The note field in all bookmarks should be aligned.

Screenshots

Desktop (please complete the following information):

• OS: MacOS X
• Browser Firefox
• Version 69.0

Server (please complete the following information):

• OS: Ubuntu 18.04
• HTTP server: Apache 2.4.38-3+deb10u1
• Database MySQL 10.4.8
• PHP version: 7.3.9
• Nextcloud version: 16.0.4.1
• Bookmarks app version: 2.1.1
• Activated Nextcloud Apps:

Enabled:
  - accessibility: 1.2.0
  - activity: 2.9.1
  - admin_audit: 1.6.0
  - announcementcenter: 3.5.1
  - apporder: 0.7.1
  - bookmarks: 2.1.1
  - bruteforcesettings: 1.4.0
  - calendar: 1.7.1
  - camerarawpreviews: 0.7.0
  - circles: 0.17.7
  - cloud_federation_api: 0.2.0
  - comments: 1.6.0
  - contacts: 3.1.3
  - cookbook: 0.4.2
  - cospend: 0.1.0
  - dashboard: 6.0.0
  - dashboardcharts: 0.0.1
  - dav: 1.9.2
  - deck: 0.6.6
  - drawio: 0.9.3
  - dropit: 0.2.1
  - event_update_notification: 0.3.4
  - external: 3.3.0
  - extract: 1.2.0
  - federatedfilesharing: 1.6.0
  - federation: 1.6.0
  - files: 1.11.0
  - files_accesscontrol: 1.6.0
  - files_automatedtagging: 1.6.0
  - files_downloadactivity: 1.5.0
  - files_ebookreader: 0.0.1
  - files_external: 1.7.0
  - files_mindmap: 0.0.15
  - files_pdfviewer: 1.5.0
  - files_photospheres: 1.0.5
  - files_readmemd: 1.1.1
  - files_retention: 1.5.1
  - files_rightclick: 0.15.1
  - files_sharing: 1.8.0
  - files_texteditor: 2.8.0
  - files_trackdownloads: 1.5.0
  - files_trashbin: 1.6.0
  - files_versions: 1.9.0
  - files_videoplayer: 1.5.0
  - firstrunwizard: 2.5.0
  - forms: 1.0.3
  - gallery: 18.3.0
  - groupfolders: 4.1.0
  - guests: 1.1.1
  - impersonate: 1.3.0
  - keeporsweep: 0.2.1
  - logreader: 2.1.0
  - lookup_server_connector: 1.4.0
  - mail: 0.17.0
  - maps: 0.1.2
  - metadata: 0.9.0
  - music: 0.10.1
  - news: 13.1.6
  - nextcloud_announcements: 1.5.0
  - notes: 3.0.2
  - notifications: 2.4.1
  - oauth2: 1.4.2
  - ocr: 4.4.6
  - ocsms: 2.1.3
  - ownbackup: 19.5.0
  - ownpad: 0.6.13
  - passman: 2.3.2
  - password_policy: 1.6.0
  - passwords: 2019.9.0
  - phonetrack: 0.5.2
  - polls: 0.10.2
  - previewgenerator: 2.1.0
  - privacy: 1.0.0
  - provisioning_api: 1.6.0
  - qownnotesapi: 19.4.0
  - quicknotes: 0.1.9
  - quota_warning: 1.5.0
  - radio: 0.6.5
  - rainloop: 6.0.3
  - ransomware_detection: 0.5.2
  - ransomware_protection: 1.4.0
  - recommendations: 0.4.0
  - richdocuments: 3.4.1
  - serverinfo: 1.6.0
  - sharebymail: 1.6.0
  - sharingpath: 0.0.1
  - socialsharing_diaspora: 1.0.4
  - socialsharing_email: 1.0.5
  - socialsharing_facebook: 1.0.4
  - socialsharing_twitter: 1.0.4
  - spreed: 6.0.4
  - support: 1.0.0
  - survey_client: 1.4.0
  - systemtags: 1.6.0
  - talk_simple_poll: 1.0.0
  - tasks: 0.11.2
  - terms_of_service: 1.2.2
  - theming: 1.7.0
  - twofactor_backupcodes: 1.5.0
  - unsplash: 1.1.3
  - updatenotification: 1.6.0
  - user_usage_report: 1.1.2
  - video_converter: 0.1.0
  - viewer: 1.1.0
  - w2g2: 2.2.8
  - workflow_pdf_converter: 1.1.0
  - workflow_script: 1.1.0
  - workflowengine: 1.6.0
Disabled:
  - encryption
  - files_antivirus
  - user_ldap

• Nextcloud configuration:

{
    "system": {
        "htaccess.RewriteBase": "\/",
        "memcache.local": "\\OC\\Memcache\\APCu",
        "apps_paths": [
            {
                "path": "\/var\/www\/html\/apps",
                "url": "\/apps",
                "writable": false
            },
            {
                "path": "\/var\/www\/html\/custom_apps",
                "url": "\/custom_apps",
                "writable": true
            }
        ],
        "memcache.distributed": "\\OC\\Memcache\\Redis",
        "memcache.locking": "\\OC\\Memcache\\Redis",
        "redis": {
            "host": "***REMOVED SENSITIVE VALUE***",
            "port": 6379
        },
        "instanceid": "***REMOVED SENSITIVE VALUE***",
        "passwordsalt": "***REMOVED SENSITIVE VALUE***",
        "secret": "***REMOVED SENSITIVE VALUE***",
        "trusted_domains": [
            "nextcloud.paulcalabro.com",
            "172.31.46.200"
        ],
        "datadirectory": "***REMOVED SENSITIVE VALUE***",
        "dbtype": "mysql",
        "version": "16.0.4.1",
        "overwrite.cli.url": "http:\/\/nextcloud.paulcalabro.com",
        "overwriteprotocol": "https",
        "trusted_proxies": "***REMOVED SENSITIVE VALUE***",
        "dbname": "***REMOVED SENSITIVE VALUE***",
        "dbhost": "***REMOVED SENSITIVE VALUE***",
        "dbport": "",
        "dbtableprefix": "oc_",
        "mysql.utf8mb4": true,
        "dbuser": "***REMOVED SENSITIVE VALUE***",
        "dbpassword": "***REMOVED SENSITIVE VALUE***",
        "installed": true,
        "maintenance": false,
        "log_type": "file",
        "loglevel": 0,
        "updater.release.channel": "production",
        "ldapIgnoreNamingRules": false,
        "ldapProviderFactory": "OCA\\User_LDAP\\LDAPProviderFactory",
        "app_install_overwrite": [
            "folderplayer"
        ],
        "theme": "",
        "debug": false,
        "app.mail.imaplog.enabled": true,
        "app.mail.smtplog.enabled": true,
        "app.mail.imap.timeout": 60,
        "app.mail.smtp.timeout": 60,
        "mail_from_address": "***REMOVED SENSITIVE VALUE***",
        "mail_smtpmode": "smtp",
        "mail_sendmailmode": "smtp",
        "mail_domain": "***REMOVED SENSITIVE VALUE***",
        "mail_smtphost": "***REMOVED SENSITIVE VALUE***",
        "mail_smtpport": "25",
        "mail_smtpauth": 1,
        "mail_smtpname": "***REMOVED SENSITIVE VALUE***",
        "mail_smtppassword": "***REMOVED SENSITIVE VALUE***",
        "mail_smtpauthtype": "LOGIN",
        "mail_smtpsecure": "tls"
    }
}

• Nextcloud external user backend: none

Additional context
N/A

Web server error log
N/A

Nextcloud log (nextcloud/data/nextcloud.log)
N/A

Browser log
N/A

[marcelklehr]

Thanks for your feedback! The goal was actually not to create a table-like experience, but even in this case thee result doesn't look very nice...

[paulcalabro]

@marcelklehr Would you be okay with me making a PR for the change or would you prefer the existing view to remain the same?

[marcelklehr]

Oh, i didn't mention it in the comment, but I fixed this in https://github.com/nextcloud/bookmarks/commit/8dd0a1e8217be8988fe6854bacce06a059181b11 ;)

[paulcalabro]

Nice! Thanks for doing that! :-)

[paulcalabro]

@marcelklehr Nice work! Looks much better! 😃