Added page about responsive behavior
ClosedPublic

Authored by fabianr on Aug 23 2018, 8:23 AM.

Details

Summary

Added a page about responsive behavior to complment the page about convergence.

Diff Detail

Repository
R985 KDE Human Interface Guidelines
Branch
responsive
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 2482
Build 2500: arc lint + arc unit
fabianr requested review of this revision.Aug 23 2018, 8:23 AM
fabianr created this revision.

Excellent article!
I only have a few language-related comment, I fully agree with the content.
Great work!

source/introduction/responsive.rst
12–16

This sentence is quite long.
Suggestion for splitting it:

Responsiveness ​is not supposed to be used as a replacement for different UI and UX for different ​interaction methods (mouse, touch, pointer, remote, ...) or different ​form factors like mobiles and televisions. Instead, it means to adapt to different screen sizes in the same form factor class, like different resolutions in ​desktops and laptops.

29

grid like -> grid-like

32

directions

33

row-like

42

horizontally

fabianr updated this revision to Diff 40728.Aug 31 2018, 5:50 AM
  • Integrated phab feedback, speeling and grammar improvments
ngraham requested changes to this revision.Aug 31 2018, 8:57 PM
ngraham added a subscriber: ngraham.

Awesome stuff. Just a few comments:

source/introduction/responsive.rst
8

means, that -> means that

10

is not -> It is not

14

Maybe also add a section about adjusting to the window size, like Discover does. Something like:

different resolutions in desktops and laptops. -> different window sizes and screen resolutions.

22

for -> of

28

Another possible item:

  • Multi-column navigation layouts can show more or fewer columns at once
32

elements, like a menu bar, only -> elements such as menu bars only

41

shrinking horizontally only -> only in the horizontal direction

43

mind, that -> mind that

This revision now requires changes to proceed.Aug 31 2018, 8:57 PM
fabianr updated this revision to Diff 40963.Sep 4 2018, 8:32 AM
fabianr marked 8 inline comments as done.
  • Addressed feedback from phabricator
fabianr marked 5 inline comments as done.Sep 4 2018, 8:33 AM
ngraham accepted this revision.Sep 5 2018, 3:20 AM

Just fix one little typo then ship it!

source/introduction/responsive.rst
32

elements, elements -> elements

This revision is now accepted and ready to land.Sep 5 2018, 3:20 AM
fabianr closed this revision.Sep 5 2018, 6:24 AM