Opened 11 years ago

Last modified 10 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 11 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 11 years ago.
Patch that makes the scroll arrows bigger
scroll_arrows_new.png (14.3 KB) - added by manuq 11 years ago.
Screenshot with the patch applied.
0003-Make-notebook-arrows-white-SL-4185.patch (925 bytes) - added by manuq 11 years ago.
Patch to make the arrows white.
scroll_arrows_new-white.png (14.3 KB) - added by manuq 11 years ago.
Screenshot with white arrows.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 11 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 11 years ago by humitos

  • Keywords triage removed

Changed 11 years ago by manuq

Current state of the scroll arrows.

Changed 11 years ago by manuq

Patch that makes the scroll arrows bigger

Changed 11 years ago by manuq

Screenshot with the patch applied.

comment:3 Changed 11 years ago by manuq

  • Keywords r? added

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

Changed 11 years ago by manuq

Patch to make the arrows white.

Changed 11 years ago by manuq

Screenshot with white arrows.

comment:4 follow-up: Changed 11 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 11 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 11 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 11 years ago by manuq

Pushed scroll arrows fixes: e38a8726 and ff8e0ffe .

comment:8 Changed 11 years ago by manuq

  • Keywords r? removed

comment:9 Changed 10 years ago by dnarvaez

  • Milestone 1.0 deleted

Milestone 1.0 deleted

Note: See TracTickets for help on using tickets.