Project

General

Profile

Actions
Copy link

Bug #10908

closed

Select items with keys of a WSuggestionPopup using WBootstrap5Theme partially working

Added by Nicolas Slusarenko about 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Roel Standaert
Target version:
4.9.2
Start date:
09/03/2022
Due date:
% Done:

100%

Estimated time:

Description

There is an excellent example featuring WSuggestionPopup in

wt-4.8.0/examples/feature/suggestionpopup/

It is possible to choose a suggestion using the keys: up, down and enter.
However, if I replace this line:

setCssTheme("polished");

by these,

auto bootstrapTheme = std::make_shared<Wt::WBootstrap5Theme>();
setTheme(bootstrapTheme);

The focus on the suggested items does not change as I press up and down.
However, the WSuggestionPopup keeps track of the keys up and down because once I press enter, the selected item is correct.
I would appreciate your help in solving this problem.
Thanks!

Nicolas

Actions
Copy link
 #1

Updated by Nicolas Slusarenko about 2 years ago

To use WBootstrap5Theme I added the line

#include

I tested with Bootstrap 3 Theme replacing previous lines with,

    auto bootstrapTheme = std::make_shared<Wt::WBootstrapTheme>();
    bootstrapTheme->setVersion(Wt::BootstrapVersion::v3);
    bootstrapTheme->setResponsive(true);
    setTheme(bootstrapTheme);
    useStyleSheet("resources/themes/bootstrap/3/bootstrap-theme.min.css");

Adding this line,

#include

The focus on the suggestions change with the keys up and down, exactly as with Polished theme.
The focus does not change with Bootstrap 5.
Thanks,

Nicolas

Actions
Copy link
 #2

Updated by Roel Standaert about 2 years ago

  • Status changed from New to Analysis
  • Target version set to 4.9.0
Actions
Copy link
 #3

Updated by Roel Standaert about 2 years ago

  • Status changed from Analysis to New
Actions
Copy link
 #4

Updated by Roel Standaert about 2 years ago

  • Target version changed from 4.9.0 to 4.10.0
Actions
Copy link
 #5

Updated by Roel Standaert over 1 year ago

I see that the first issue already is that at some point we're checking APP.theme (which is not defined), and instead should be checking WT.theme.

That doesn't quite fix everything, though.

Actions
Copy link
 #6

Updated by Roel Standaert over 1 year ago

  • Status changed from New to InProgress
  • Assignee set to Roel Standaert
  • Target version changed from 4.10.0 to 4.9.2
Actions
Copy link
 #7

Updated by Roel Standaert over 1 year ago

  • Status changed from InProgress to Review
  • Assignee deleted (Roel Standaert)
Actions
Copy link
 #8

Updated by Roel Standaert over 1 year ago

  • Status changed from Review to Resolved
Actions
Copy link
 #9

Updated by Roel Standaert over 1 year ago

  • Assignee set to Roel Standaert
Actions
Copy link
 #10

Updated by Roel Standaert over 1 year ago

  • % Done changed from 0 to 100
Actions
Copy link
 #11

Updated by Roel Standaert over 1 year ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF