Opened on 09/06/2017 at 12:23:01 PM

Closed on 09/11/2019 at 02:37:07 PM

#5629 closed change (duplicate)

Fix all edge cases caused by flexible root element for new options page

Reported by: saroyanm Assignee:
Priority: P3 Milestone:
Module: User-Interface Keywords:
Cc: greiner, juliandoucette Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description

Background

After several design iterations we finally have a final version of the new options page (See #5158), but we currently are using fixed root element size.

What to change

Be sure that removing flexible root element do not cause issues in layout (e.g. buttons and tabs overflow in sidebar) and make it flexible again.

Attachments (0)

Change History (6)

comment:1 follow-up: Changed on 09/06/2017 at 01:15:28 PM by greiner

Are we sure that this is the expected behavior?

While the spec defines it as `0.3em + 13.2em + 46.3em + 0.3em`, it's unclear whether it's expected to occupy the entire space or whether we should merely center all the UI elements horizontally in which case the width would remain unchanged.

comment:2 in reply to: ↑ 1 Changed on 09/06/2017 at 01:45:27 PM by saroyanm

Replying to greiner:

Are we sure that this is the expected behavior?

While the spec defines it as `0.3em + 13.2em + 46.3em + 0.3em`, it's unclear whether it's expected to occupy the entire space or whether we should merely center all the UI elements horizontally in which case the width would remain unchanged.

I don't think that the width is the issue (if I understand your comment correctly) in this case, but I agree that we should find out final behavior of the sidebar.

The only issue currently I encounter is that the Buttons aligned to the bottom of the sidebar being overflown on the tabs which aren't aligned vertically.

comment:3 Changed on 09/26/2017 at 06:44:35 PM by saroyanm

  • Blocking 5158 removed

comment:4 Changed on 09/09/2019 at 03:12:10 PM by greiner

@saroyanm Can we close this ticket? If you still think that we need to address this concern, feel free to create a GitLab issue for it instead.

comment:5 Changed on 09/10/2019 at 03:08:30 PM by saroyanm

So far it seems like most of the elements are responsive already, the issue mainly is with the table list, Let's followup on that here -> https://gitlab.com/eyeo/adblockplus/abpui/adblockplusui/issues/579#note_214895060

comment:6 Changed on 09/11/2019 at 02:37:07 PM by greiner

  • Resolution set to duplicate
  • Status changed from new to closed

Ok, thanks. In that case I'm closing this ticket in favor of ui#579.

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
 
Note: See TracTickets for help on using tickets.