Opened 7 years ago

Last modified 6 years ago

#4185 new defect

Number of tabs

Reported by: erikos Owned by: manuq
Priority: Low Milestone:
Component: Browse Version: Unspecified
Severity: Minor Keywords:
Cc: garycmartin Distribution/OS: Unspecified
Bug Status: New

Description

Build: os11

In Browse you can add as many tabs as you want. After a while you get some arrows that allows you to scroll through the list. With touch those are hard to trigger. Furthermore I wonder if opening so many tabs is beneficial to the user.

One option would be to limit the number of tabs you can do. If n is reached the add-tab button could be made insensitive. n should be calculated on the screen width (taking into account rotation of the screen).

Attachments (5)

scroll_arrows_current.png (16.5 KB) - added by manuq 7 years ago.
Current state of the scroll arrows.
0002-Notebook-make-scroll-arrows-2.5-times-bigger-for-tou.patch (1.4 KB) - added by manuq 7 years ago.
Patch that makes the scroll arrows bigger
scroll_arrows_new.png (14.3 KB) - added by manuq 7 years ago.
Screenshot with the patch applied.
0003-Make-notebook-arrows-white-SL-4185.patch (925 bytes) - added by manuq 7 years ago.
Patch to make the arrows white.
scroll_arrows_new-white.png (14.3 KB) - added by manuq 7 years ago.
Screenshot with white arrows.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 7 years ago by humitos

  • Bug Status changed from Unconfirmed to New
  • Milestone changed from Unspecified by Release Team to 1.0
  • Priority changed from Unspecified by Maintainer to Low
  • Severity changed from Unspecified to Minor

Move out from 0.98 after discussion on IRC.

comment:2 Changed 7 years ago by humitos

  • Keywords triage removed

Changed 7 years ago by manuq

Current state of the scroll arrows.

Changed 7 years ago by manuq

Patch that makes the scroll arrows bigger

Changed 7 years ago by manuq

Screenshot with the patch applied.

comment:3 Changed 7 years ago by manuq

  • Keywords r? added

Made the scroll arrows a bit bigger. Further description is in the patch commit message.

Changed 7 years ago by manuq

Patch to make the arrows white.

Changed 7 years ago by manuq

Screenshot with white arrows.

comment:4 follow-up: Changed 7 years ago by manuq

Setting the arrows color to white looks better. Added another patch and screenshot.

comment:5 in reply to: ↑ 4 ; follow-up: Changed 7 years ago by garycmartin

Replying to manuq:

Setting the arrows color to white looks better. Added another patch and screenshot.

Yes, white looks much better than black. I take it that the triangle shape is not easily modified to match our Sugary navigation svg's (as used in the bottom tray areas of Record and Browse)?

comment:6 in reply to: ↑ 5 Changed 7 years ago by manuq

Replying to garycmartin:

Replying to manuq:

Setting the arrows color to white looks better. Added another patch and screenshot.

Yes, white looks much better than black. I take it that the triangle shape is not easily modified to match our Sugary navigation svg's (as used in the bottom tray areas of Record and Browse)?

Hey Gary, thanks for taking a look. You are right about the arrows shape. We need a custom theme for that, see #4090 . Hopefully we will do that soon, then all our arrows will get the original shape.

comment:7 Changed 7 years ago by manuq

Pushed scroll arrows fixes: e38a8726 and ff8e0ffe .

comment:8 Changed 7 years ago by manuq

  • Keywords r? removed

comment:9 Changed 6 years ago by dnarvaez

  • Milestone 1.0 deleted

Milestone 1.0 deleted

Note: See TracTickets for help on using tickets.